#1405: TOVSolver: do not set conservatives, remove TOV_Atmosphere -----------------------------------+---------------------------------------- Reporter: rhaas | Owner: Type: enhancement | Status: new Priority: minor | Milestone: Component: EinsteinToolkit thorn | Version: development version Keywords: TOVSolver | -----------------------------------+---------------------------------------- the attached two patches remove TOV_Atmosphere[] from the code and do not set the conservative variables. Instead TOVSolver relies on GRHydro to set the conservatives in InitialPrim2Con and to enforce atmosphere in initialatmospherereset. This also fixes any asymmetry in atmosphere handling between the stars.
Most tests pass without change (since GRHydro ran anyway), the failing one being the test_two_av TOVSolver test which the second one regenerates. Differences are small but above threshold in the average case.
#1405: TOVSolver: do not set conservatives, remove TOV_Atmosphere ------------------------------------+--------------------------------------- Reporter: rhaas | Owner: Type: enhancement | Status: review Priority: minor | Milestone: Component: EinsteinToolkit thorn | Version: development version Resolution: | Keywords: TOVSolver ------------------------------------+--------------------------------------- Changes (by rhaas):
* status: new => review
#1405: TOVSolver: do not set conservatives, remove TOV_Atmosphere ------------------------------------+--------------------------------------- Reporter: rhaas | Owner: Type: enhancement | Status: reviewed_ok Priority: minor | Milestone: Component: EinsteinToolkit thorn | Version: development version Resolution: | Keywords: TOVSolver ------------------------------------+--------------------------------------- Changes (by knarf):
* status: review => reviewed_ok
Comment:
The patch looks fine. Please apply
#1405: TOVSolver: do not set conservatives, remove TOV_Atmosphere ------------------------------------+--------------------------------------- Reporter: rhaas | Owner: Type: enhancement | Status: closed Priority: minor | Milestone: Component: EinsteinToolkit thorn | Version: development version Resolution: fixed | Keywords: TOVSolver ------------------------------------+--------------------------------------- Changes (by rhaas):
* status: reviewed_ok => closed * resolution: => fixed
Comment:
committed as revs 143 and 144 of TOVSolver.
trac@lists.einsteintoolkit.org