Mercurial > repos > public > sbplib_julia

diff test/SbpOperators/volumeops/derivatives/second_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/second_derivative_test.jl	Tue Nov 29 21:25:07 2022 +0100
+++ b/test/SbpOperators/volumeops/derivatives/second_derivative_test.jl	Tue Nov 29 21:46:54 2022 +0100
@@ -6,9 +6,8 @@
 
 import Sbplib.SbpOperators.VolumeOperator
 
-# Review:
-# If we test LazyTensor.inflation for e.g 1D-3D general tensors then we should only need to test
-# the 1D second derivative.
+# TODO: Refactor these test to look more like the tests in first_derivative_test.jl.
+
 @testset "SecondDerivative" begin
     operator_path = sbp_operators_path()*"standard_diagonal.toml"
     stencil_set = read_stencil_set(operator_path; order=4)