-
Notifications
You must be signed in to change notification settings - Fork 45
IOU error #20
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
Comments
So there are some datasets that use class 255 to label pixels that are ambiguous. These pixels should not count towards IOU calculation. In utils.py I first made sure they are uint8, and then add 1 to the tensor, which will make these 255 to 0. Then these pixels with label 0 (originally labeled 255) are filtered out. Let me know if this explanation aligns with the code? |
Yes your explanation aligns with the code. But I think we should not do like this. |
I see your concern, but I think another good property of the metric would be: 0.0 for the worst algorithm in the world, and 1.0 for the best algorithm in the world. If I understand your proposal correctly, for datasets with any ambiguous pixel, this metric can never reach 1.0? To me this is not ideal. Another important aspect is to align with prior implementations, so that the numbers are directly comparable. I believe I checked this implementation against PASCAL calculation and ADE20K calculation. |
Thank you, now i got it |
Thank you so much for sharing!
But maybe I found an error in utils.py.
When computing IOU, you filtered the pred where mask == 0. It will make the IOU higher.(line 47)
Is it correct?
The text was updated successfully, but these errors were encountered: