Mercurial > repos > public > sbplib_julia

diff test/SbpOperators/volumeops/derivatives/first_derivative_test.jl @ 1158:c94a12327737 refactor/sbpoperators/inflation

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
Disregard review comments about multi-d tests. After discussion on Discord we concluded that removing multi-d tests for volume operators would make them too implementation dependent.
author Jonatan Werpers <jonatan@werpers.com>
date Tue, 29 Nov 2022 21:46:54 +0100
parents f1bb1b6d85dd
children 7d52c4835d15
line wrap: on
line diff
--- a/test/SbpOperators/volumeops/derivatives/first_derivative_test.jl	Tue Nov 29 21:25:07 2022 +0100
+++ b/test/SbpOperators/volumeops/derivatives/first_derivative_test.jl	Tue Nov 29 21:46:54 2022 +0100
@@ -19,9 +19,7 @@
     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)