New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

(radiation) control flux limiter with switch; read additional radiation quantities from dumps #406

Merged

danieljprice merged 1 commit into master from radiation

Apr 23, 2023

Collaborator

themikelau commented Apr 23, 2023

Type of PR:

Bug fix + feature

Description:

Explicit radiative diffusion now selects whether or not to use the (Levermore & Pomraning) flux limiter depending on the input option limit_radiation_flux. Previous behaviour was to always use an optically-thick diffusion coefficient (lambda->1/3), which is a bug. Can @danieljprice check if this change is safe for implicit radiative diffusion?
readwrite_dumps modified to also read grad(E) and lambda from a dump that used radiation.


          (radiation) use control flux limiter with flag; read additional radia…

5612f12

…tion quantities from dumps

themikelau requested a review from danieljprice

April 23, 2023 09:47

danieljprice merged commit 29790d8 into master

danieljprice deleted the radiation branch

April 23, 2023 23:06

Owner

danieljprice commented Apr 23, 2023

all looks good to me

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet