#1266: Featue request for Parity Symmetry thorn ---------------------------------+------------------------------------------ Reporter: yosef@… | Owner: Type: enhancement | Status: new Priority: minor | Milestone: Component: Other | Version: Keywords: | ---------------------------------+------------------------------------------ I started looking at runs with parity symmetry, that is the only symmetry is the simultaneous reflection about x, y, and z. I think I was able to "implement" this symmetry by changing just a handful of lines in Rotating180 and I'm a little less sure about my changes to carpetregrid2. I was wondering, would there be any interest in adding this symmetry to ET?
#1266: Featue request for Parity Symmetry thorn ----------------------------------+----------------------------------------- Reporter: yosef@… | Owner: Type: enhancement | Status: new Priority: minor | Milestone: Component: Other | Version: Resolution: | Keywords: ----------------------------------+-----------------------------------------
Comment (by yosef@…):
The files par_rot_diff is the diff between the new ParitySymmetry thorn and RotatingSymmetry180. carpet_diff is the diff between the standard version of carpetRegrid2 and the version consistent with the symmetry.
#1266: Featue request for Parity Symmetry thorn ----------------------------------+----------------------------------------- Reporter: yosef@… | Owner: Type: enhancement | Status: new Priority: minor | Milestone: Component: Other | Version: Resolution: | Keywords: ----------------------------------+-----------------------------------------
Comment (by rhaas):
I am not sure I understand what the difference between the parity symmetry you describe and thorn reflectionsymmetry with reflect_x = reflect_y = reflect_z = yes is.
#1266: Featue request for Parity Symmetry thorn ----------------------------------+----------------------------------------- Reporter: yosef@… | Owner: Type: enhancement | Status: new Priority: minor | Milestone: Component: Other | Version: Resolution: | Keywords: ----------------------------------+-----------------------------------------
Comment (by anonymous):
Replying to [comment:2 rhaas]:
I am not sure I understand what the difference between the parity
symmetry you describe and thorn reflectionsymmetry with reflect_x = reflect_y = reflect_z = yes is.
The system is not x-reflection, y-reflection, or z-reflection symmetric. Rather, the only symmetry is (x, y, z) -> (-x, -y, -z). I.e., this is a bitant symmetry where we use (x,y,-z) -> (-x, -y, +z) to fill in the z<0 ghostzones.
#1266: Featue request for Parity Symmetry thorn ----------------------------------+----------------------------------------- Reporter: yosef@… | Owner: Type: enhancement | Status: new Priority: minor | Milestone: Component: Other | Version: Resolution: | Keywords: ----------------------------------+-----------------------------------------
Comment (by yosef@…):
I placed the paritysymmetry thorn in incoming. The patch to carpetregir2 will be needed for AMR runs.
Yosef
#1266: Feature request for Parity Symmetry thorn ----------------------------------+----------------------------------------- Reporter: yosef@… | Owner: Type: enhancement | Status: new Priority: minor | Milestone: Component: Other | Version: Resolution: | Keywords: ----------------------------------+-----------------------------------------
#1266: Feature request for Parity Symmetry thorn ----------------------------------+----------------------------------------- Reporter: yosef@… | Owner: Type: enhancement | Status: new Priority: minor | Milestone: Component: Other | Version: Resolution: | Keywords: ----------------------------------+-----------------------------------------
Comment (by rhaas):
carpet_diff applies and does not break the tests. However it's comments are all referring to rotatingsymmetry180. Once these are fixed to refer to parity symmetry, that patch can be applied.
The new thorn ParitySymmetry lacks documentation and test cases. It cannot be part of the ET in this form unfortunately. Otherwise having such a symmetry would be welcome.
#1266: Feature request for Parity Symmetry thorn ----------------------------------+----------------------------------------- Reporter: yosef@… | Owner: Type: enhancement | Status: new Priority: minor | Milestone: Component: Other | Version: Resolution: | Keywords: ----------------------------------+-----------------------------------------
Comment (by yosef@…):
Here's an updated patch for CarpetRegrid2. The testsuite for ParitySymmetry is in Incoming.
#1266: Feature request for Parity Symmetry thorn ----------------------------------+----------------------------------------- Reporter: yosef@… | Owner: Type: enhancement | Status: new Priority: minor | Milestone: Component: Other | Version: Resolution: | Keywords: ----------------------------------+-----------------------------------------
Comment (by rhaas):
carpet_diff2 is fine to apply (will do so later today unless there are objections).
#1266: Feature request for Parity Symmetry thorn ----------------------------------+----------------------------------------- Reporter: yosef@… | Owner: Type: enhancement | Status: review Priority: minor | Milestone: Component: Other | Version: Resolution: | Keywords: ----------------------------------+----------------------------------------- Changes (by rhaas):
* status: new => review
Comment:
On 2013-11-12 10:34, Yosef Zlochower wrote:> I was wondering about this patch. I don't see it
in the latest development version. Can it go in after the new release (I'm assuming you don't want to add a new feature before the new release)?
Indeed. Too late to apply before the release. Can be applied afterwards, only useful with Yosef's Parity thorn as well (the thorn is in incoming I believe).
#1266: Feature request for Parity Symmetry thorn ----------------------------------+----------------------------------------- Reporter: yosef@… | Owner: Type: enhancement | Status: reviewed_ok Priority: minor | Milestone: Component: Other | Version: Resolution: | Keywords: ----------------------------------+----------------------------------------- Changes (by rhaas):
* status: review => reviewed_ok
#1266: Feature request for Parity Symmetry thorn ----------------------------------+----------------------------------------- Reporter: yosef@… | Owner: Type: enhancement | Status: reviewed_ok Priority: minor | Milestone: Component: Other | Version: Resolution: | Keywords: ----------------------------------+-----------------------------------------
Comment (by rhaas):
Just checked. The thorn is in incoming, but not in a shape to include in the ET. See comment:6 it also lists eg Erik in the README file as an author (since it is a copy of RotatingSymmetry180), most likely Yosef should be added to properly attribute credit (and blame).
#1266: Feature request for Parity Symmetry thorn ----------------------------------+----------------------------------------- Reporter: yosef@… | Owner: Type: enhancement | Status: closed Priority: minor | Milestone: Component: Other | Version: Resolution: fixed | Keywords: ----------------------------------+----------------------------------------- Changes (by rhaas):
* status: reviewed_ok => closed * resolution: => fixed
Comment:
Applied the change to CarpetRegrid2 in Carpet hash 4d72bda475a2330da5973bd71c9f23e5291b3cf0 "CarpetRegrid2: support Parity Symmetry thorn". For the thorn itself, please open a separate ticket.
trac@lists.einsteintoolkit.org
-
Einstein Toolkit