-
Notifications
You must be signed in to change notification settings - Fork 4
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
1D2V simulations with electric field switch #268
Comments
I'm not sure, but maybe it could belong in the composition section of the input file, considering this would be entirely different physics from kinetic or fluid ions? So I guess one of the new flags for |
I think this is orthogonal to all other options if one just sets a flag that is used in em_fields.jl to set |
Sounds much more sensible! |
I'd be tempted to add this as an |
I view this as a physics hack for debugging and testing rather than a self-consistent model. The present version of the PR would let you calculate everything everywhere, just with Ez= Er = 0 #269. However, if you are sure that this is a physical limit, then I have no objection to your suggestion to make this as an electron model. Perhaps |
I think as a physical limit it only makes sense, as John says, if we make electrons massless first, and then take their temperature to zero. So it is a limit in the boltzmann electron approximation and as long as that's clear then it's fine. Otherwise, it's a strange regime, where electron mean flow is larger than their thermal flow and even the negatively charged wall assumption stops making sense. |
Looking more carefully at the existing code in master, I think that the effect of #269 might be achieved by just setting
@MantasAbazorius if you have a moment to check the output from one of your runs using #269 against an input file where you just set |
The ion distribution functions are identical for both T_e=0.0 and the #269 option. When checking the rest of the dynamic data, as expected, mismatches are in data that depends on T_e: electron_thermal_speed, chodura_integral_upper, chodura_integral_lower, electron_parallel_pressure and phi. |
OK, I will close the PR #269 as redundant. @MantasAbazorius if you make a new install, use master branch from now on, with
|
To support Mantas Abazorius's work, it would be useful to implement a switch to turn off
Ez
so that the DKE solved for is@johnomotani @LucasMontoya4 Where should I implement the switch? Do you agree that this could be an interesting feature?
The text was updated successfully, but these errors were encountered: