-
Notifications
You must be signed in to change notification settings - Fork 5
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
Add netCDF support V2 #34
Conversation
Codecov Report
@@ Coverage Diff @@
## master #34 +/- ##
==========================================
- Coverage 94.29% 91.77% -2.52%
==========================================
Files 11 12 +1
Lines 1315 1484 +169
Branches 281 318 +37
==========================================
+ Hits 1240 1362 +122
- Misses 50 89 +39
- Partials 25 33 +8
Continue to review full report at Codecov.
|
If we want other reviewers, we could always ping IAMconsortium/pyam#335 |
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.
Huge @lewisjared this looks great!
Pull request
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:
Requires:
Supercedes #26
Closes #15