Mercurial > repos > public > sbplib_julia

diff test/SbpOperators/volumeops/derivatives/first_derivative_test.jl @ 1153:f1bb1b6d85dd refactor/sbpoperators/inflation

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Review: Suggest changes to test and removal of conveninece constructor
author Vidar Stiernström <vidar.stiernstrom@it.uu.se>
date Tue, 25 Oct 2022 10:33:27 +0200
parents 157a78959e5d
children c94a12327737
line wrap: on
line diff
--- a/test/SbpOperators/volumeops/derivatives/first_derivative_test.jl	Fri Oct 21 23:07:30 2022 +0200
+++ b/test/SbpOperators/volumeops/derivatives/first_derivative_test.jl	Tue Oct 25 10:33:27 2022 +0200
@@ -19,7 +19,9 @@
     end
     x^k/factorial(k)
 end
-
+# Review:
+# If we test LazyTensor.inflation for e.g 1D-3D general tensors then we should only need to test
+# the 1D first derivative.
 @testset "first_derivative" begin
     @testset "Constructors" begin
         stencil_set = read_stencil_set(sbp_operators_path()*"standard_diagonal.toml"; order=2)