search

Cartucho / OpenLabeling

Label images and video for Computer Vision applications
Apache License 2.0
926 stars 265 forks source link

Adding the ability to change the label of the selected box #29

Closed seanthegreat7 closed 5 years ago

seanthegreat7 commented 5 years ago

If you select a box using keys 'f' and 'r' you can change the label of the selected box the same way that 'w' and 's' work without drawing it again.

Cartucho commented 5 years ago

Looks great! However, don't you think it would be better if it used the same keys as usual w and s instead of adding new keys?

To avoid any conflict everytime someone selects a bounding-box the selected class should change to the one of that bounding box.

What do you think?

Cartucho commented 5 years ago

Hello Sean!

Sorry for the late response. It's much better with the w and s!

There are still some bugs, for example, if you press the key w many times when changing a bounding boxes' label it will eventually result in an IndexError: index out of range. You can solve this in the function change_selected_bbox(c) you should check if the img_index is in the right range.

Also could you please update to the latest version with a git pull?