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 support for CMIC product from PPSv2021 #1992
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1992 +/- ##
==========================================
+ Coverage 93.54% 93.64% +0.10%
==========================================
Files 279 282 +3
Lines 41322 41823 +501
==========================================
+ Hits 38654 39165 +511
+ Misses 2668 2658 -10
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
@ninahakansson feel free to have a look! |
unzip.return_value = '' | ||
self.scn = NcNWCSAF('filename', {}, {}) | ||
self.filehandler_class = NcNWCSAF | ||
self.scn = self.filehandler_class('filename', {}, {}) |
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.
Can this property be renamed to avoid confusion? I was already confused when you set self.scn.nc
in one of the tests since that isn't something a Scene should have.
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.
ah yes, good point
The CPP product has been renamed in NWCSAF 2021 to CMIC, and even if the datasets inside are the same, the name of the variables in the nc files have been changed. This PR allows reading either file transparently by introducing file_keys and prefixes to the nwcsaf-nc reader.
Moreover, the default composites for these products have been updated.