[Einstein Toolkit] #1309: implement UIUC's speed-up in "evaluation" of spectral solution
#1309: implement UIUC's speed-up in "evaluation" of spectral solution --------------------------------------+------------------------------------- Reporter: bernard.j.kelly@… | Owner: Type: enhancement | Status: new Priority: major | Milestone: Component: EinsteinToolkit thorn | Version: ET_2012_11 Keywords: twopunctures evaluation | --------------------------------------+------------------------------------- The accompanying svn patch applies the recently released refinement of TwoPunctures by Vasileios Paschalidis and Zach Etienne at UIUC, greatly reducing the time taken to properly apply the solution of the spectral solve to all grid points.
The patch is relative to the ET_2012_11 release (though I suspect it would apply to the upcoming 2013_05 release without modification). It passes the TwoPunctures/test/bhns_eval test suite.
#1309: implement UIUC's speed-up in "evaluation" of spectral solution ---------------------------------------+------------------------------------ Reporter: bernard.j.kelly@… | Owner: Type: enhancement | Status: new Priority: major | Milestone: Component: EinsteinToolkit thorn | Version: ET_2012_11 Resolution: | Keywords: twopunctures evaluation ---------------------------------------+------------------------------------
Comment (by hinder):
Thanks for doing the merge! Yes, there has only been one commit to TwoPunctures since the release, and this was to an unrelated file:
http://git.barrywardell.net/arrangements/EinsteinInitialData/TwoPunctures.gi...
Since we are now in feature-freeze, this will have to wait until after the release, but it should be one of the first things that we review once that is done.
#1309: implement UIUC's speed-up in "evaluation" of spectral solution ---------------------------------------+------------------------------------ Reporter: bernard.j.kelly@… | Owner: Type: enhancement | Status: review Priority: major | Milestone: Component: EinsteinToolkit thorn | Version: ET_2012_11 Resolution: | Keywords: twopunctures evaluation ---------------------------------------+------------------------------------ Changes (by rhaas):
* status: new => review
#1309: implement UIUC's speed-up in "evaluation" of spectral solution ---------------------------------------+------------------------------------ Reporter: bernard.j.kelly@… | Owner: Type: enhancement | Status: reviewed_ok Priority: major | Milestone: Component: EinsteinToolkit thorn | Version: ET_2012_11 Resolution: | Keywords: twopunctures evaluation ---------------------------------------+------------------------------------ Changes (by rhaas):
* status: review => reviewed_ok
Comment:
Patch looks fine. Only thing to '''possibly''' add is an option to switch from the old to the new method. If I remembers this all correctly then the results of the "fast" method should be bit-identical to those of the "slow" method since it is just a storing of intermediate results. Please apply.
#1309: implement UIUC's speed-up in "evaluation" of spectral solution ---------------------------------------+------------------------------------ Reporter: bernard.j.kelly@… | Owner: Type: enhancement | Status: reviewed_ok Priority: major | Milestone: Component: EinsteinToolkit thorn | Version: ET_2012_11 Resolution: | Keywords: twopunctures evaluation ---------------------------------------+------------------------------------
Comment (by rhaas):
Note: once someone comments on the added option, I am fine to apply this with proper "from..." comments (since Bernard most likely cannot, not can Vasilis).
#1309: implement UIUC's speed-up in "evaluation" of spectral solution ---------------------------------------+------------------------------------ Reporter: bernard.j.kelly@… | Owner: Type: enhancement | Status: reviewed_ok Priority: major | Milestone: Component: EinsteinToolkit thorn | Version: ET_2012_11 Resolution: | Keywords: twopunctures evaluation ---------------------------------------+------------------------------------
Comment (by hinder):
If the results are the same, even within a tolerance (and this is tested by the existing test case, I think), then there should be no need to add the parameter.
I just looked at the test cases. There is no vacuum test that uses grid_setup_method = evaluation. I don't understand how the hydro initial data works in the BHNS tests in TwoPunctures/test; maybe these tests are sufficient to test the whole of TwoPunctures?
#1309: implement UIUC's speed-up in "evaluation" of spectral solution ---------------------------------------+------------------------------------ Reporter: bernard.j.kelly@… | Owner: Type: enhancement | Status: reviewed_ok Priority: major | Milestone: Component: EinsteinToolkit thorn | Version: ET_2012_11 Resolution: | Keywords: twopunctures evaluation ---------------------------------------+------------------------------------
Comment (by hinder):
On second thought, I would be happier if the tests were testing more than just what happens on the x axis and in a norm. For something like this, I would expect several points tested in all directions. For example, 3D ascii output (probably downsampled from the full grid) of at least u and gxx.
#1309: implement UIUC's speed-up in "evaluation" of spectral solution ---------------------------------------+------------------------------------ Reporter: bernard.j.kelly@… | Owner: Type: enhancement | Status: reviewed_ok Priority: major | Milestone: Component: EinsteinToolkit thorn | Version: ET_2012_11 Resolution: | Keywords: twopunctures evaluation ---------------------------------------+------------------------------------
Comment (by eschnett):
It should be easy to set up a small 3D grid covering only a small region near a horizon, e.g. 4^3 grid points. Full 3D ASCII output should then work quite well.
#1309: implement UIUC's speed-up in "evaluation" of spectral solution ---------------------------------------+------------------------------------ Reporter: bernard.j.kelly@… | Owner: Type: enhancement | Status: closed Priority: major | Milestone: Component: EinsteinToolkit thorn | Version: ET_2012_11 Resolution: fixed | Keywords: twopunctures evaluation ---------------------------------------+------------------------------------ Changes (by rhaas):
* status: reviewed_ok => closed * resolution: => fixed
Comment:
Applied as rev 133 of TwoPunctures. We do actually have a test that uses evaluation, in the bhns_eval test of TOVSolver. The results before and after the patch differ only in the last bit.
trac@lists.einsteintoolkit.org
-
Einstein Toolkit