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
Use more generalized keyword argument for gyroradius #1210
Use more generalized keyword argument for gyroradius #1210
Conversation
This commit replaces T_i paramter keyword with T for more generalization. Adds a deprectaion warning for notifying users.
Could someone please give this PR a review. |
Codecov Report
@@ Coverage Diff @@
## main #1210 +/- ##
=======================================
Coverage ? 96.98%
=======================================
Files ? 71
Lines ? 7030
Branches ? 0
=======================================
Hits ? 6818
Misses ? 212
Partials ? 0 Continue to review full report at Codecov.
|
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.
Looks good to me! With a few slight edits it'll be ready to merge.
We use pre-commit to check that the code is formatted as expected by black
and isort
. We just added an option to implement these formatting fixes by including the following in a GitHub comment:
pre-commit.ci autofix
pre-commit.ci autofix |
Closes #1170
Waiting for feedback on initial changes. Will proceed further with tests.
docstrings.