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
Update Clavrx reader for netcdf files #1716
Conversation
…ation Fix donor nav name to catch new l1b_base in attributes
Codecov Report
@@ Coverage Diff @@
## main #1716 +/- ##
==========================================
+ Coverage 92.43% 92.83% +0.40%
==========================================
Files 258 260 +2
Lines 38273 38552 +279
==========================================
+ Hits 35376 35791 +415
+ Misses 2897 2761 -136
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
The PR updates the current CLAVRX HDF reader to include a netcdf reader for geostationary files currently being produced. Closes #1661 |
…a does not get flagged as a polar swath)
…not get assigned a polar swath
…er confirming from area definiton
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.
A couple small changes, otherwise this looks pretty good to me. You mentioned you were worried about using the utilities classes correctly. Was there something specific you wanted me to look at?
Update copyright Move code into _CLAVRxHelper that handles data reading and scaling
The PR updates the current CLAVRX HDF reader to include a netcdf reader for geostationary files currently being produced.