-
Notifications
You must be signed in to change notification settings - Fork 18
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
Implement fast recycling #176
Conversation
- Very simple expression at the moment, need to check later whether there are any factors missing from a detailed derivation - Unsure whether neutral energy source is energy or pressure. Previously neutral energy was T_FC * N and added to energy_source, which seems inconsistent.
The fast recycling energy wasn't being added to the actual energy source, only to the diagnostic energy source
There is an issue here. The formula for the energy of recycled neutrals is using a weighting of temperature, not heat flux. Yes, the returning thermal neutrals are all 3eV and their heat flux calculation is therefore very straightforward. However, the ions are a drifting maxwellian going through Probably the easiest way would be to add it to |
- This allows the heat flow through the sheath to be used by other parts of the code
Field energy_flow_ylow should be shifted to non-aligned coordinates. In axisymmetric simulations this doesn't make a difference. Thanks Ben!
Heat flow now correctly calculated from ion sheath heat flow which is calculated in sheath_boundary_simple. sheath_boundary not yet supported. Improved variable names and comments for clarity
Further improvements to code clarity and readability
Corrected the formulation to use the actual ion energy flows leaving the model, extended to SOL/PFR, fixed some bugs and improved code clarity. Documentation next |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Commit fe6161c causes a segmentation fault in the 1D-recycling integrated test.
@bendudson ready for review. Unfortunately it seems to arbitrarily fail the documentation build test again. I have no idea why since all I've changed is some text... it would be good to finally get this merged and reduce the amount of live PRs. Do you think the issue is with the test or with the documentation itself? This is the error:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @mikekryjak ! Readthedocs is acting up at the moment, will sort out at some point.
Turns out fast recycling has a big impact. This PR does a simple implementation of this:
Where$R$ is the recycle multiplier, $R_{f}$ is the fast reflection fraction, $\alpha_{E}$ is the energy reflection factor,
$\Gamma_{E_{i}}^{sheath}$ is the incident heat flux from the sheath boundary condition, $T_{R}$ is the recycle energy and $\Gamma_{N_{i}}$ is the incident ion particle flux.
This is implemented for the SOL and the PFR and is fully compatible with the neutral pump.
In addition, the example cases haven't been corrected to reflect the new recycling flags - this is now done in this PR for all examples.
Tasks: