-
Notifications
You must be signed in to change notification settings - Fork 77
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 values_dtype backend option to load values at full precision #407
Conversation
@iainrussell What are your thoughts on the updated PR? |
Ping @iainrussell |
Cc @SF-N |
Hi @juntyr, sorry for the long delay, but I've got back to this now - your PR looks good, I just added a couple of tests and plan to make a release tomorrow. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #407 +/- ##
==========================================
+ Coverage 41.34% 41.85% +0.50%
==========================================
Files 26 26
Lines 2201 2222 +21
Branches 194 194
==========================================
+ Hits 910 930 +20
- Misses 1291 1292 +1 ☔ View full report in Codecov by Sentry. |
Thank you very much for your help @iainrussell! |
cfgrib 0.9.15.0 has been released with this new feature - thank you @juntyr! |
Supersedes #395
#395 (comment)