-
Notifications
You must be signed in to change notification settings - Fork 300
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
ISCCP-NG level 1g reader #3002
base: main
Are you sure you want to change the base?
ISCCP-NG level 1g reader #3002
Conversation
Also sun zenith angles only contains one layer of data.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3002 +/- ##
========================================
Coverage 96.08% 96.09%
========================================
Files 377 379 +2
Lines 55155 55255 +100
========================================
+ Hits 52997 53097 +100
Misses 2158 2158
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Pull Request Test Coverage Report for Build 12083858536Details
💛 - Coveralls |
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.
Looks good I think. One question inline, but also:
Looks like there is no area definition generated by this reader. Would it be possible to have one? in general, areas are easier and faster than lon/lat-based swath definitions…
def get_dataset(self, key, yaml_info): | ||
"""Get dataset.""" | ||
logger.debug("Getting data for: %s", yaml_info["name"]) | ||
nc = xr.open_dataset(self.filename, chunks={"y": CHUNK_SIZE, "x": 800}) |
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.
Instead of using the legacy chunk size, how about using auto
?
This is a reader for the ISCCP-NG L1g format https://cimss.ssec.wisc.edu/isccp-ng/. Nominal calibration us used for SEVIRI. It might be good to recalibrate SEVIRI. But I can not find information in the file on the coefficients used. So these would need to be supplied separately.