-
Notifications
You must be signed in to change notification settings - Fork 303
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 MERSI3 / FY3F support. #2838
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2838 +/- ##
==========================================
- Coverage 95.95% 95.95% -0.01%
==========================================
Files 366 366
Lines 53617 53615 -2
==========================================
- Hits 51448 51446 -2
Misses 2169 2169
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
You forgot to change the filetype name from mersi2 to mersi3... |
Haha, well spotted - fixed now. Thanks @yukaribbba! |
Pull Request Test Coverage Report for Build 10002879545Details
💛 - 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.
LGTM, is this ready to merge?
Ah, I see it is ready, merging |
This PR adds support for the MERSI-3 instrument aboard FY-3F. There's no code changes required, as the existing MERSI-2 code can be used, so only new YAML files are added.