Closed mraspaud closed 3 months ago
Attention: Patch coverage is 87.50000%
with 1 line
in your changes missing coverage. Please review.
Project coverage is 95.78%. Comparing base (
2bd607a
) to head (ad353f0
). Report is 349 commits behind head on main.
Files with missing lines | Patch % | Lines |
---|---|---|
satpy/readers/sar_c_safe.py | 87.50% | 1 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
This is marked as enhancement, should it be bug fix? It is a "compatibility" fix technically right?
I was seeing it as an enhancement as it is not a bug in satpy... but I'm not picky, bug works if you prefer it.
Yeah it could be seen either way. I guess it depends what you want it released as.
This PR introduces a fix for the backwards incompatibility introduced in https://github.com/corteva/rioxarray/pull/787, and thus should make the tests for SAR pass again.