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
Fix naming #311
Fix naming #311
Conversation
@lewisjared sorry about the black changes included here... I wonder if now is the time to bite the bullet and switch things across to RCMIP names, or do we think that will cause too much chaos in downstream uses? |
Actually let's not do it everywhere now, chaos really would ensue. One for pymagicc 3.0, can be tracked with in #312. |
Codecov Report
@@ Coverage Diff @@
## master #311 +/- ##
==========================================
- Coverage 93.47% 93.31% -0.17%
==========================================
Files 18 18
Lines 2483 2482 -1
Branches 483 483
==========================================
- Hits 2321 2316 -5
- Misses 103 105 +2
- Partials 59 61 +2
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.
LGTM. One decision left:
- 'Surface Temperature' to 'Surface Air Temperature Change'
I'd keep it out of this PR and deal with that pain later. It isn't 100% RCMIP, but close enough...
Pull request
Fixes some of the naming to bring us closer to RCMIP conventions
Please confirm that this pull request has done the following:
CHANGELOG.rst
addedAdding to CHANGELOG.rst
Please add a single line in the changelog notes similar to one of the following: