#481: Tests should be converted to use new symmetry thorns -----------------------------------+---------------------------------------- Reporter: hinder | Owner: Type: defect | Status: new Priority: major | Milestone: Component: EinsteinToolkit thorn | Version: Keywords: | -----------------------------------+---------------------------------------- It was decided (http://cactuscode.org/pipermail/developers/2010-September/006087.html) that the built-in CartGrid3D symmetries are deprecated and will not work with the Mercurial version of Carpet. Any test cases that use Grid::domain != "full" should be modified to use ReflectionSymmetry instead.
In general, this should be possible without regenerating the test output. However, for the case of AHFinderDirect, the horizon finder actually uses a different internal multipatch grid structure if it detects that a CartGrid3D symmetry is being used. This means that the horizon shapes and quantities are different at the level of numerical error with ReflectionSymmetry. I propose to convert the tests to use ReflectionSymmetry and regenerate the test data (the grid function phi is the same with the new symmetry thorn, it's only the horizons which are different). The failing tests are related to checkpoint and recovery, and are not designed to test the symmetry mechanism.
#481: Tests should be converted to use new symmetry thorns ------------------------------------+--------------------------------------- Reporter: hinder | Owner: Type: defect | Status: new Priority: major | Milestone: Component: EinsteinToolkit thorn | Version: Resolution: | Keywords: ------------------------------------+---------------------------------------
Comment (by eschnett):
This is a good idea.
#481: Tests should be converted to use new symmetry thorns ------------------------------------+--------------------------------------- Reporter: hinder | Owner: hinder Type: defect | Status: accepted Priority: major | Milestone: Component: EinsteinToolkit thorn | Version: Resolution: | Keywords: ------------------------------------+--------------------------------------- Changes (by hinder):
* owner: => hinder * status: new => accepted
#481: Tests should be converted to use new symmetry thorns ------------------------------------+--------------------------------------- Reporter: hinder | Owner: hinder Type: defect | Status: closed Priority: major | Milestone: Component: EinsteinToolkit thorn | Version: Resolution: fixed | Keywords: ------------------------------------+--------------------------------------- Changes (by rhaas):
* status: accepted => closed * resolution: => fixed
Comment:
No such tests (that is runable) exist anymore. The only ones using the old style symmetries use PUGH. AHFinderDirect was changed to use new symmetries.
{{{ cd TEST/sim for i in */*/ ; do t=${i#*/} ; grep -Hi '::domain>' ../../arrangements/*/${i%/*/}/test/${t%/}.par* | gawk 'BEGIN{FS=":"}!/full/{print $1}' ; done | grep -li carpet }}}
trac@lists.einsteintoolkit.org
-
Einstein Toolkit