-
Notifications
You must be signed in to change notification settings - Fork 17
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: 8bit/16bit, big-endian/little-endian in nexrad reader #231
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #231 +/- ##
==========================================
- Coverage 92.80% 92.78% -0.02%
==========================================
Files 26 26
Lines 5003 5005 +2
==========================================
+ Hits 4643 4644 +1
- Misses 360 361 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@syedhamidali Could you please test your workflow with these changes? We would have to create a release branch for the 0.7-line and backport this. Otherwise users are forced to upgrade to latest xarray. This might take some time to figure out the workflows, though. |
@kmuehlbauer Yes looks better now. Thank you! |
…#231) * fix 8bit/16bit, big-endian/little-endian in nexrad reader * only apply mask for 2 byte data * add history.md entry
history.md