#1068: GRHydro: schedule GRHydro_Tmunu* only if there is Tmunu storage. ------------------------------------+--------------------------------------- Reporter: bmundim | Owner: Type: defect | Status: review Priority: major | Milestone: Component: EinsteinToolkit thorn | Version: Resolution: | Keywords: GRHydro Tmunu storage ------------------------------------+---------------------------------------
Comment (by rhaas):
The patch itself looks ok. I am not sure though if this is required at all. TmunuBase itself already contains a statement: {{{ if (stress_energy_storage) { ... SCHEDULE GROUP AddToTmunu IN SetTmunu AFTER (TmunuBase_SetTmunu,TmunuBase_ZeroTmunu) { } "Add to the stress-energy tensor here" ... } }}} so that the whole AddToTmunu group is never scheduled unless stress_energy_state is true (or at least unless the parameter value that will be copied into stress_energy_state is true).