Open ameraner opened 2 weeks ago
Attention: Patch coverage is 98.03922%
with 1 line
in your changes missing coverage. Please review.
Project coverage is 95.94%. Comparing base (
834f45d
) to head (85f11a3
). Report is 6 commits behind head on main.
Files | Patch % | Lines |
---|---|---|
satpy/enhancements/__init__.py | 92.85% | 1 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
satpy/enhancements/init.py | 17 | 18 | 94.44% | ||
<!-- | Total: | 34 | 35 | 97.14% | --> |
Totals | |
---|---|
Change from base Build 9497248893: | 0.002% |
Covered Lines: | 51589 |
Relevant Lines: | 53715 |
Hi Dave, thanks for the review! Indeed, at the beginning I thought of this only as an improved interface from satpy to the Colorbar
, but then I also realised that this functionality is probably useful and better places in trollimage directly.
So I created a new method in trollimage: https://github.com/pytroll/trollimage/pull/170
I updated this PR accordingly, but kept the tests for the new functionality here, and and of course they're failing now since we need the trollimage PR released first.
After the released update in https://github.com/pytroll/trollimage/pull/170, and the according change in the required trollimage version, the tests related to this PR are passing again as expected, so from that point of view I believe this PR is ready.
However, now there are failures on the sar_c_safe
reader tests that are unrelated to this PR...
The SAR tests broke because of backwards incompatibility introduced in rioxarray release 0.15.6 and in particular this PR https://github.com/corteva/rioxarray/pull/787 I suggest blacklisting rioxarray 0.15.6 for now, until we figure a solution with @snowman2 on that PR
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
satpy/enhancements/init.py | 13 | 14 | 92.86% | ||
<!-- | Total: | 30 | 31 | 96.77% | --> |
Files with Coverage Reduction | New Missed Lines | % | ||
---|---|---|---|---|
satpy/readers/olci_nc.py | 8 | 94.5% | ||
<!-- | Total: | 8 | --> |
Totals | |
---|---|
Change from base Build 9596553856: | 0.002% |
Covered Lines: | 51616 |
Relevant Lines: | 53742 |
Tests passing again after blacklisting rioxarray 0.15.6
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
satpy/enhancements/init.py | 13 | 14 | 92.86% | ||
<!-- | Total: | 30 | 31 | 96.77% | --> |
Totals | |
---|---|
Change from base Build 9687731633: | 0.009% |
Covered Lines: | 51642 |
Relevant Lines: | 53768 |
I reverted the rioxarray blacklist after Martin's fix, and tests pass again now after Dave's fixes yesterday.
So from my side this is good to go again.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.
Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
---|---|---|---|---|---|
satpy/enhancements/init.py | 13 | 14 | 92.86% | ||
<!-- | Total: | 50 | 51 | 98.04% | --> |
Totals | |
---|---|
Change from base Build 9687731633: | 0.01% |
Covered Lines: | 51668 |
Relevant Lines: | 53794 |
This PR adds the possibility to define an alpha channel range to be applied to a colorise colormap, e.g. through the yaml enhancement configuration.
With this, composites like the one below are more easily achievable, with e.g.