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
Fix label2rgb #4661
Fix label2rgb #4661
Conversation
please do not merge as I need to add tests and also it seems that the problem is not completely solved |
Hello @emmanuelle! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found: There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻 Comment last updated at 2020-05-08 09:58:38 UTC |
ok this is now ready for review :-) |
@@ -38,6 +38,17 @@ def test_wrong_kind(): | |||
label2rgb(label, kind='foo', bg_label=-1) | |||
|
|||
|
|||
def test_uint_image(): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this test is failing on master, img was not rescaled with img_as_float
and therefore kept a 0-255 range while the color mask to be added was in the [0-1] range so not visible when superimposed. The output was float but in [0-255] so also matplotlib would represent it completely white in the gallery examples.
Closes #4657