Castro icon indicating copy to clipboard operation
Castro copied to clipboard

revisit the transverse update

Open zingale opened this issue 3 years ago • 3 comments

Right now, we do the transverse correction to the conserved state + p. This way we don't need to use the EOS to get p on the interface when we go back to the primitive state for the Riemann solve.

But the EOS is pretty cheap now (with the templating), that it might be cleaner and more robust to update the conserved system, derive (rho e) after the correction and handle it in the same dual energy fashion as we do elsewhere, and then use ctoprim to get the new corrected primitive edge state.

zingale avatar Jun 01 '21 01:06 zingale

I think that this would actually match what we do with MHD

zingale avatar Jun 01 '21 01:06 zingale

this is implemented in #1920

zingale avatar Jun 25 '21 17:06 zingale

note that the bug fix on the reset species with density was done separately now in #2120

zingale avatar Apr 23 '22 19:04 zingale