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.

Sign up for GitHub

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

Jump to bottom

Fix label_id division by 256 -> 255 #2455

Merged

k-okada merged 1 commit into jsk-ros-pkg:master from wkentaro:label_image_decomposer_division

Oct 21, 2019

Member

wkentaro commented Oct 19, 2019 •

edited

Loading

Since len(colormap) is 255, % 256 is wrong. It can return 255, which raises IndexError.


          Fix label_id division by 256 -> 255

699b734

Since `len(colormap)` is `255`, % 256 is wrong since it can return 255
which raises IndexError.

wkentaro added the bug label

knorth55 approved these changes

View reviewed changes

k-okada merged commit 94ea19e into jsk-ros-pkg:master

wkentaro deleted the label_image_decomposer_division branch

October 21, 2019 08:49

This was referenced Oct 1, 2020

jsk_recognition: 1.2.11-1 in 'noetic/distribution.yaml' [bloom] ros/rosdistro#26772

Merged

jsk_recognition: 1.2.11-1 in 'melodic/distribution.yaml' [bloom] ros/rosdistro#26773

Merged

jsk_recognition: 1.2.11-1 in 'kinetic/distribution.yaml' [bloom] ros/rosdistro#26774

Merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug