BOUT-dev icon indicating copy to clipboard operation
BOUT-dev copied to clipboard

Add unit test for LaplaceXZPetsc

Open dschwoerer opened this issue 4 years ago • 3 comments

dschwoerer avatar Jul 01 '21 09:07 dschwoerer

See #2207 for some discussion

ZedThree avatar Jul 01 '21 10:07 ZedThree

Possible directions for investigation:

  • convergence checks for the round-trip test here: if the issue is one of mismatched discretisations, the error should decrease as we increase the resolution
  • analytic tests: this could be done in both uniform and spatially-dependent metric tensors by solving the forward operator with something like sympy or mathematica -- this would give us analytic tests for both forward and inverse operators

ZedThree avatar Jul 07 '21 09:07 ZedThree

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] avatar Nov 16 '22 13:11 github-actions[bot]