[Users] Hydro_RNSID Inherits from GRHydro

Steven R. Brandt sbrandt at cct.lsu.edu
Mon Nov 5 13:27:19 CST 2018

On 11/5/18 12:33 PM, Roland Haas wrote:
> Hello all,
>> Hydro_RNSID inherits from GRHydro, which goes against the usual policy that
>> initial data thorns depend only on HydroBase / ADMBase. Is there a plan to
>> fix this? Is a fix even difficult?
> There's a pull request (8 months old) that fixes this here:
> https://trac.einsteintoolkit.org/ticket/2128
> Since I am the author I cannot review it, but anyone else can.

You can, however, nag people and ask repeatedly about reviews on the 
mailing list.


> Note that this *could* induce a slight change in variables since
> GRHydro's prim2con may not quite compute the conserved variables in the
> very same way that Hydro_RNSID computed the primitives.
> We *could* get perfect backwards compatibility by adding a parameter
> and having Hydro_RNSID do what it does now if the parameter is set to
> "yes" (which would be the default). One has to get the variable
> pointers from VarDataPtr and probably should check for GRHydro being
> active in ParamCheck.
> Yours,
> Roland
> _______________________________________________
> Users mailing list
> Users at einsteintoolkit.org
> http://lists.einsteintoolkit.org/mailman/listinfo/users
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.einsteintoolkit.org/pipermail/users/attachments/20181105/1ec6e2ae/attachment.html 

More information about the Users mailing list