[Einstein Toolkit] #816: include SphericalHarmonicReconASCII from incoming in PITTNULLCode

List overview All Threads
Download

newer

older

[Einstein Toolkit] #1797: allow...

[Einstein Toolkit] #1799: Missing...

Einstein Toolkit

19 Apr 2012 19 Apr '12

1:59 p.m.

#816: include SphericalHarmonicReconASCII from incoming in PITTNULLCode -----------------------------------+---------------------------------------- Reporter: rhaas | Owner: Type: enhancement | Status: new Priority: optional | Milestone: Component: EinsteinToolkit thorn | Version: Keywords: | -----------------------------------+---------------------------------------- Christian Reisswig provided a copy of SphericalHarmonicReconASCII and alternative thorn that reads in CCE boundary data (similar SphericalHarmonicRecon) but supporting a wider variety of input file formats (HDF5 among them, irrespective of the name).

Used by SpEC and Llama.

There are no docs or test cases as of now. Test data would be welcome (SpEC, Llama or Cactus provided, I don't think it makes a difference).

-- Ticket URL: https://trac.einsteintoolkit.org/ticket/816 Einstein Toolkit http://einsteintoolkit.org The Einstein Toolkit

Show replies by date

Einstein Toolkit

19 Apr 19 Apr

3:38 p.m.

#816: include SphericalHarmonicReconASCII from incoming in PITTNULLCode ------------------------------------+--------------------------------------- Reporter: rhaas | Owner: Type: enhancement | Status: new Priority: optional | Milestone: Component: EinsteinToolkit thorn | Version: Resolution: | Keywords: ------------------------------------+---------------------------------------

Comment (by eschnett):

Where are these thorns?

-- Ticket URL: https://trac.einsteintoolkit.org/ticket/816#comment:1 Einstein Toolkit http://einsteintoolkit.org The Einstein Toolkit

Einstein Toolkit

4:02 p.m.

Comment (by rhaas):

PITTNull is part of the ET already:

https://svn.einsteintoolkit.org/cactus/PITTNullCode

SphericalHarmonicReconASCII is incoming/PITTNullCode:

https://svn.einsteintoolkit.org/incoming/PITTNullCode/SphericalHarmonicRecon...

-- Ticket URL: https://trac.einsteintoolkit.org/ticket/816#comment:2 Einstein Toolkit http://einsteintoolkit.org The Einstein Toolkit

Einstein Toolkit

20 Apr 20 Apr

8:21 a.m.

Comment (by eschnett):

This thorn lacks documentation, examples, and test cases.

Instead of scheduling SphericalHarmonicReconASCII_Startup both at initial and post_recover_variables, I recommend scheduling only at basegrid.

-- Ticket URL: https://trac.einsteintoolkit.org/ticket/816#comment:3 Einstein Toolkit http://einsteintoolkit.org The Einstein Toolkit

Einstein Toolkit

31 Oct 31 Oct

9:36 p.m.

Comment (by rhaas):

The thorn now has test suites (data and parfile by Nick Taylor). They were generated (by me) using 2 MPI processes. However I found that they fails when run with 1 or 4 MPI processes. The tests use PUGH and output only reduction results (min,max,norm_info,norm2).

The thorn was also renamed to https://svn.einsteintoolkit.org/incoming/PITTNullCode/SphericalHarmonicRecon...

-- Ticket URL: https://trac.einsteintoolkit.org/ticket/816#comment:4 Einstein Toolkit http://einsteintoolkit.org The Einstein Toolkit

Einstein Toolkit

9:40 p.m.

Comment (by knarf):

Is it known what the cause for the failure on different numbers of processors is (tolerance too low for reduction-based tests)?

-- Ticket URL: https://trac.einsteintoolkit.org/ticket/816#comment:5 Einstein Toolkit http://einsteintoolkit.org The Einstein Toolkit

Einstein Toolkit

9:49 p.m.

Comment (by rhaas):

I have to admit I have no idea. I just noticed it tonight. Since the thorn won't make it into the release (lack of documentation) this is not terribly high on my list of things to do before the release :-). The test uses the default tolerances so RELTOL=ABSTOL=1e-12. Output seems to differ at about this level. However the maximum should be independent of the number of processes I think. {{{#!diff diff SpEC-h5-test/NewsB[0]_maximum.asc /NewsB[0]_maximum.asc

3c3 < 963.0000000000000 0.0034305367678 ---

...

963.0000000000000 0.0034305368375

}}}

-- Ticket URL: https://trac.einsteintoolkit.org/ticket/816#comment:6 Einstein Toolkit http://einsteintoolkit.org The Einstein Toolkit

Einstein Toolkit

25 Feb 25 Feb

4:30 p.m.

Comment (by rhaas):

Replying to [comment:3 eschnett]:

...

This thorn lacks documentation, examples, and test cases.

Instead of scheduling SphericalHarmonicReconASCII_Startup both at

initial and post_recover_variables, I recommend scheduling only at basegrid. Scheduling only at basedgrid will currently fail since the startup routine copies cctk_iteration into its own data structures. Since BaseGrid runs before CCTK_RECOVER_VARIABLES it would see the wrong iteration number. A solution might be to not make copies of cctkGH members in thorn-owned data structures.

-- Ticket URL: https://trac.einsteintoolkit.org/ticket/816#comment:7 Einstein Toolkit http://einsteintoolkit.org The Einstein Toolkit

Einstein Toolkit

5:38 p.m.

Comment (by eschnett):

The iteration number (and other cGH entries) should be set before recover_variables. They should be available in basegrid. Can you open a ticket for this?

-- Ticket URL: https://trac.einsteintoolkit.org/ticket/816#comment:8 Einstein Toolkit http://einsteintoolkit.org The Einstein Toolkit

Einstein Toolkit

6:04 p.m.

Comment (by rhaas):

I have to admit that I did not check what actually happens and only looked at the schedule output (CarpetIOHDF5 certainly fiddles with the iteration number though). I will investigate and open the required number of new tickets. So the idea is to set the cGH entries as early as recover_parameters?

-- Ticket URL: https://trac.einsteintoolkit.org/ticket/816#comment:9 Einstein Toolkit http://einsteintoolkit.org The Einstein Toolkit

Einstein Toolkit

6:06 p.m.

Comment (by eschnett):

Either there, or some time shortly thereafter. However, all grid functions (and, by extension, all cGH entries) must be available in basegrid. The cGH should be set up at the same time as the grid structure.

-- Ticket URL: https://trac.einsteintoolkit.org/ticket/816#comment:10 Einstein Toolkit http://einsteintoolkit.org The Einstein Toolkit

Einstein Toolkit

22 Jan 22 Jan

8:41 p.m.

Comment (by rhaas):

I just have (for the first time) actually worked with the code. I believe that it will be best to spend an afternoon and clean up the code.

TODO items:

* make sure all externally visible functions with C linkage start with "SphericalHarmonicReconGeneric_" * consolidate the (global) variables in the SHR namespace into a single source file, vars.cc * remove commented out sections of code * use CCTK_INFO for informational output instead of cout * use CCTK_Error to abort rather than assert * clean up double vs CCTK_REAL issues * terminate cleanly when running out of data in mode files

-- Ticket URL: https://trac.einsteintoolkit.org/ticket/816#comment:11 Einstein Toolkit http://einsteintoolkit.org The Einstein Toolkit

Einstein Toolkit

8 Oct 8 Oct

3:49 p.m.

#816: include SphericalHarmonicReconASCII from incoming in PITTNULLCode ------------------------------------+--------------------------------------- Reporter: rhaas | Owner: Type: enhancement | Status: confirmed Priority: minor | Milestone: ET_2014_11 Component: EinsteinToolkit thorn | Version: Resolution: | Keywords: ------------------------------------+--------------------------------------- Changes (by hinder):

* priority: optional => minor * status: new => confirmed * milestone: => ET_2014_11

Comment:

Bela has an enhancement to this thorn and would like to commit it. The current status is that this thorn lives at https://svn.einsteintoolkit.org/incoming/PITTNullCode/SphericalHarmonicRecon... but apparently has not been included in the toolkit due to a desire to clean up the code. For convenience, I think this should be imported into the main PittNullCode repository. I have created a git svn clone of the SVN repository, and imported this (as a subtree merge) into a new branch "sphericalharmonicrecongen" of the BitBucket PittNullCode repository. I have not applied the username mappings or attempted any other complicated cleanup of the branch. It would be good if someone could tidy up the code on this branch, get it reviewed, and hopefully into the next release, as it has been sitting in incoming for a very long time.

-- Ticket URL: https://trac.einsteintoolkit.org/ticket/816#comment:12 Einstein Toolkit http://einsteintoolkit.org The Einstein Toolkit

Einstein Toolkit

13 Oct 13 Oct

10:12 a.m.

#816: include SphericalHarmonicReconASCII from incoming in PITTNULLCode ------------------------------------+--------------------------------------- Reporter: rhaas | Owner: Type: enhancement | Status: confirmed Priority: minor | Milestone: ET_2014_11 Component: EinsteinToolkit thorn | Version: Resolution: | Keywords: ------------------------------------+---------------------------------------

Comment (by knarf):

Needed: a bit of cleanup, but at least some documentation.

-- Ticket URL: https://trac.einsteintoolkit.org/ticket/816#comment:13 Einstein Toolkit http://einsteintoolkit.org The Einstein Toolkit

Einstein Toolkit

10 Nov 10 Nov

9:05 a.m.

#816: include SphericalHarmonicReconASCII from incoming in PITTNULLCode ------------------------------------+--------------------------------------- Reporter: rhaas | Owner: Type: enhancement | Status: confirmed Priority: minor | Milestone: Component: EinsteinToolkit thorn | Version: Resolution: | Keywords: ------------------------------------+--------------------------------------- Changes (by knarf):

* milestone: ET_2014_11 =>

-- Ticket URL: https://trac.einsteintoolkit.org/ticket/816#comment:14 Einstein Toolkit http://einsteintoolkit.org The Einstein Toolkit

Einstein Toolkit

13 Jul 13 Jul

1:45 a.m.

#816: include SphericalHarmonicReconASCII from incoming in PITTNULLCode ------------------------------------+--------------------------------------- Reporter: rhaas | Owner: Type: enhancement | Status: review Priority: minor | Milestone: Component: EinsteinToolkit thorn | Version: Resolution: | Keywords: ------------------------------------+--------------------------------------- Changes (by rhaas):

* status: confirmed => review

Comment:

Christian Reisswig took care of all remaining issues in svn revision 213 and 214. Unless vetoed in today's call I will apply this by the end of the week of Jul 13 2015.

-- Ticket URL: https://trac.einsteintoolkit.org/ticket/816#comment:15 Einstein Toolkit http://einsteintoolkit.org The Einstein Toolkit

Einstein Toolkit

1:45 a.m.

#816: include SphericalHarmonicReconASCII from incoming in PITTNULLCode ------------------------------------+--------------------------------------- Reporter: rhaas | Owner: Type: enhancement | Status: reviewed_ok Priority: minor | Milestone: Component: EinsteinToolkit thorn | Version: Resolution: | Keywords: ------------------------------------+--------------------------------------- Changes (by rhaas):

* status: review => reviewed_ok

-- Ticket URL: https://trac.einsteintoolkit.org/ticket/816#comment:16 Einstein Toolkit http://einsteintoolkit.org The Einstein Toolkit

Einstein Toolkit

4 Aug 4 Aug

7 a.m.

#816: include SphericalHarmonicReconASCII from incoming in PITTNULLCode ------------------------------------+--------------------------------------- Reporter: rhaas | Owner: Type: enhancement | Status: closed Priority: minor | Milestone: Component: EinsteinToolkit thorn | Version: Resolution: fixed | Keywords: ------------------------------------+--------------------------------------- Changes (by rhaas):

* status: reviewed_ok => closed * resolution: => fixed

Comment:

Included in the ET thorn list in git hash 87de2d10f3296ce1a54ff7a70fc82716b5240404 of the manifest.

-- Ticket URL: https://trac.einsteintoolkit.org/ticket/816#comment:17 Einstein Toolkit http://einsteintoolkit.org The Einstein Toolkit

4084

Age (days ago)

5286

Last active (days ago)

trac@lists.einsteintoolkit.org

17 comments

1 participants

tags (0)

participants (1)

Einstein Toolkit