pytroll / satpy

Python package for earth-observing satellite data processing
http://satpy.readthedocs.org/en/latest/
GNU General Public License v3.0
1.06k stars 292 forks source link

Ici reader tiepoints bugfix #2701

Closed BengtRydberg closed 5 months ago

BengtRydberg commented 9 months ago

This small PR fixes how pixel positions are reconstructed from tiepoints for the ICI reader. The tiepoints are not evenly distributed around the scan end as was assumed earlier.

codecov[bot] commented 9 months ago

Codecov Report

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

Project coverage is 95.39%. Comparing base (4c30838) to head (2aef549). Report is 333 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #2701 +/- ## ======================================= Coverage 95.39% 95.39% ======================================= Files 371 371 Lines 52683 52693 +10 ======================================= + Hits 50256 50266 +10 Misses 2427 2427 ``` | [Flag](https://app.codecov.io/gh/pytroll/satpy/pull/2701/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pytroll) | Coverage Δ | | |---|---|---| | [behaviourtests](https://app.codecov.io/gh/pytroll/satpy/pull/2701/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pytroll) | `4.16% <0.00%> (+<0.01%)` | :arrow_up: | | [unittests](https://app.codecov.io/gh/pytroll/satpy/pull/2701/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pytroll) | `96.00% <100.00%> (+<0.01%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=pytroll#carryforward-flags-in-the-pull-request-comment) to find out more.

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