Visitar URL original
Move `ionic_levels` to `plasmapy.particles.atomic` by namurphy · Pull Request #1697 · PlasmaPy/PlasmaPy · GitHub
Skip to content

Move ionic_levels to plasmapy.particles.atomic - #1697

Merged
namurphy merged 8 commits into
PlasmaPy:mainfrom
namurphy:move-ionic-levels
Aug 26, 2022
Merged

namurphy merged 8 commits into
PlasmaPy:mainfrom
namurphy:move-ionic-levels

Conversation

@namurphy

Copy link
Copy Markdown
Member

This PR moves ionic_levels from plasmapy.particles.particle_collections to plasmapy.particles.atomic.

This PR is a spinoff of #1057. As noted in this commit, this was moved to avoid a circular import during the refactoring of particle_input.

@github-actions github-actions Bot added the plasmapy.particles Related to the plasmapy.particles subpackage label Aug 26, 2022
@namurphy
namurphy requested a review from a team August 26, 2022 00:41
@codecov

codecov Bot commented Aug 26, 2022 •

Copy link
Copy Markdown

Codecov Report

Merging #1697 (e8157f0) into main (ae2ad84) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1697      +/-   ##
==========================================
- Coverage   97.65%   97.64%   -0.01%     
==========================================
  Files          85       85              
  Lines        8128     8126       -2     
==========================================
- Hits         7937     7935       -2     
  Misses        191      191              
Impacted Files Coverage Δ
plasmapy/particles/__init__.py 95.83% <100.00%> (ø)
plasmapy/particles/atomic.py 100.00% <100.00%> (ø)
plasmapy/particles/ionization_state.py 94.04% <100.00%> (-0.02%) ⬇️
plasmapy/particles/particle_collections.py 99.16% <100.00%> (-0.06%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@github-actions github-actions Bot added the docs PlasmaPy Docs at http://docs.plasmapy.org label Aug 26, 2022
Comment thread changelog/1697.breaking.rst Resolved

@StanczakDominik StanczakDominik left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I skimmed the code, but since it's just a simple move checks passing are enough here 👍

@namurphy
namurphy merged commit 3b33c42 into PlasmaPy:main Aug 26, 2022
@namurphy
namurphy deleted the move-ionic-levels branch August 26, 2022 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs PlasmaPy Docs at http://docs.plasmapy.org plasmapy.particles Related to the plasmapy.particles subpackage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants