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
BUG: colorbar did not match normalised colors #1089
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1089 +/- ##
==========================================
- Coverage 90.2% 88.25% -1.95%
==========================================
Files 19 19
Lines 1837 1839 +2
==========================================
- Hits 1657 1623 -34
- Misses 180 216 +36
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #1089 +/- ##
==========================================
+ Coverage 89.67% 89.71% +0.03%
==========================================
Files 20 20
Lines 1967 1973 +6
==========================================
+ Hits 1764 1770 +6
Misses 203 203
Continue to review full report at Codecov.
|
Probably resolves #393, but that issue is super unclear. |
Fixed the conflict after merging the other PR |
This is giving some errors. It might be that the test needs to be updated now the other PR is merged. |
Tests should be fixed now. I'll wait for CI and merge. |
In current version, if you use normalisation, colorbar does not represent true color scheme as it is always normalized linear between min and max. This PR fixes the behaviour and normalises colorbar using the same scheme, so the results should match.
It should resolve issue like #697 (comment) .
Before:
After:
Closes #697