- changed status to open
- removed comment
provide names for spherical surfaces
the attached set of patches provide a simple function
sf_IdFromName(CCTK_INT id, CCTK_STRING name) (and similar for Fortran) as well as changes for AHFinderDirect, QuasiLocalMeasures and SetMask_SphericalSurface to use it.
The code compiles and passes the testsuites for QLM and AHFinderDirect (testing both with ids by name and by number).
If this code is roughly in concordance with how this ought-to-be-done in Cactus, then please apply (I have commit rights to QLM and AHFinderDirect only but not SphericalSurface and SetMask_SphericalSurface).
Keyword:
Comments (3)
-
-
reporter - removed comment
I' added a check for uniqueness. I just re-checked which repositories I can write to (by checking in which there is a commit by rhaas). It seems I need only permissions for QuasiLocalMeasures really. I'll ask Frank for permission since it is in LSUThorns. Function SetMask_SphericalSurface is in a thorn EinsteinUtils/SetMask_SphericalSurface by Frank Löffler. As far as I can tell it does more or less what CarpetMask's excluded_surface parameter does.
-
reporter - changed status to resolved
- removed comment
- Log in to comment
I like this idea.
In your example, please use something else but "SphericalSurface0" as surface name -- if this were the road to go, then names don't have any advantage...
You may want to add parameter checking in SphericalSurface to ensure that all surfaces have distinct names.
Please apply. Please ask for commit rights to SphericalSurface to do this. Which thorn contains SetMask_SphericalSurface?