satpy icon indicating copy to clipboard operation
satpy copied to clipboard

Add MERSI3 / FY3F support.

Open simonrp84 opened this issue 1 year ago • 1 comments

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.

simonrp84 avatar Jun 25 '24 14:06 simonrp84

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 95.95%. Comparing base (b62f10b) to head (1832175). Report is 452 commits behind head on main.

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              
Flag Coverage Δ
behaviourtests 4.04% <ø> (+<0.01%) :arrow_up:
unittests 96.05% <ø> (-0.01%) :arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Jun 25 '24 14:06 codecov[bot]

You forgot to change the filetype name from mersi2 to mersi3...

yukaribbba avatar Jul 10 '24 15:07 yukaribbba

You forgot to change the filetype name from mersi2 to mersi3...

Haha, well spotted - fixed now. Thanks @yukaribbba!

simonrp84 avatar Jul 11 '24 06:07 simonrp84

Pull Request Test Coverage Report for Build 10002879545

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 96.054%

Totals Coverage Status
Change from base Build 9988332527: 0.0%
Covered Lines: 51675
Relevant Lines: 53798

💛 - Coveralls

coveralls avatar Jul 19 '24 05:07 coveralls

Ah, I see it is ready, merging

mraspaud avatar Jul 19 '24 10:07 mraspaud