-
Notifications
You must be signed in to change notification settings - Fork 339
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
Mag reynolds #820
Mag reynolds #820
Conversation
(#PlasmaPy 815)
Hello @KhalilBryant! Thanks for updating your pull request. Congratulations! There are no PEP8 issues in this pull request. 😸 Comment last updated at 2020-05-31 18:02:47 UTC |
Codecov Report
@@ Coverage Diff @@
## master #820 +/- ##
=======================================
Coverage 95.87% 95.88%
=======================================
Files 56 56
Lines 5117 5124 +7
=======================================
+ Hits 4906 4913 +7
Misses 211 211
Continue to review full report at Codecov.
|
I merged the PR for Reynolds, could you merge those changes in? I squashed them into a single commit so as to make the git history tree less verbose. |
# Conflicts: # plasmapy/formulary/dimensionless.py # plasmapy/formulary/tests/test_dimensionless.py
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.
LGTM!
changelog/README.rst
for instructions, and if you need help with picking the PR type, ask!)docstrings. (Tests pop up at the bottom, in the checks box).
they're failing, ask!).
This Pull request partially solves Create functionality for calculating dimensionless plasma parameters #49 and covers part of New dimensionless parameter prandtl number #769
It also merged all of Reynolds #815 with it