8000 B) Chained, no edge flux by mikekryjak · Pull Request #1 · mikekryjak/hermes-3 · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

B) Chained, no edge flux #1

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.

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

Open
wants to merge 18 commits into
base: master
Choose a base branch
from
Open

B) Chained, no edge flux #1

wants to merge 18 commits into from

Conversation

mikekryjak
Copy link
Owner
@mikekryjak mikekryjak commented Oct 4, 2023

For testing purposes only

Has latest mas 10000 ter with latest fast recycling: boutproject@818976d

And has chained limiters before further updates: boutproject@c8a7051

Now it also has the ability to have separate conduction, convection and momentum limiters.

bendudson and others added 18 commits August 8, 2023 21:44
Trying to get model closer to Horsten thesis (2019).
The net effect of these changes is to:
- Increase heat conduction by 5/3
- Increase heat advection by 5/3
- Increase viscosity by 5/2
- Add viscous heating

The atomic rates used in diffusion, viscosity, and heat conduction
coefficients are still different from Horsten 2019.
Convert from viscous heating power to rate of change of pressure.
Intended to be similar to Wim Van Uytven et al "Assessment of advanced
fluid neutral models for the neutral atoms in the plasma edge and
application in ITER geometry" Nucl. Fusion 62 (2022) 086023

Significantly slows convergence; may be something wrong in implementation.
Previous commit accidentally reversed the sign of the cross-field
particle flow, resulting in very poor convergence.

Flux limits are now enabled by default.
- This is for testing and the Rnn term may be removed permanently later
- The AFN limiters don't have this.
Check if float option is < 0 rather than equal to -1.
Remove some unused and commented-out code.
Flux limited particle flux is used in momentum flux before the
momentum flux is limited. Particle flux limiter applied to advection
of momentum and pressure.
Applies a time-averaging to the flux limit factors, with a given
timescale. Doesn't appear to have much benefit in quick tests, but
left in and turned off by default. May be removed if it really doesn't
do anything useful.
@mikekryjak
Copy link
Owner Author

Updated description to clarify that latest fast recycling is included

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0