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
Decorate two_fluid
dispersion solver with @particle_input
#2022
Decorate two_fluid
dispersion solver with @particle_input
#2022
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #2022 +/- ##
==========================================
- Coverage 98.31% 98.31% -0.01%
==========================================
Files 98 98
Lines 8622 8608 -14
==========================================
- Hits 8477 8463 -14
Misses 145 145
☔ View full report in Codecov by Sentry. |
The charge number is now contained in `ion`.
two_fluid
with @particle_input
two_fluid
dispersion solver with @particle_input
…article-input-two-fluid
The ones from that are positional are all distinguishable by unit or particle-like-ness, so having allowing it to be positional is not likely to cause any problems.
…article-input-two-fluid
…article-input-two-fluid
…article-input-two-fluid
…article-input-two-fluid
z_mean
is provided
#2178
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.
👍 could users possible get confused between charge state and z mean?
My hope is that by implementing Thank you as always for the reviews! |
Description
This PR decorates
plasmapy.dispersion.analytical.two_fluid_.two_fluid
with@particle_input
.A new capability is that
two_fluid
will now acceptmass_numb
separately fromion
.There are some breaking changes:
z_mean
parameter got changed toZ
fortwo_fluid
(for consistency with how we're handling the charge numberz_mean
will no longer override the charge provided inion
(since having contradictory arguments make code confusing to read without referring to the docstring)Motivation and context
Using
@particle_input
here will allow us to drop two blocks of code which were validatingion
andz_mean
. Plus, using the mechanics allowed by@particle_input
will give a more consistent user experience across subpackages.Related issues
This is motivated by #1869, which at present is marking some
z_mean
override tests as expecting to fail.This is an offshoot of #341, except for
plasmapy.dispersion
instead ofplasmapy.formulary
.