Cartucho / OpenLabeling

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

Can't delete box if created bottom right point first #7

Closed MattKleinsmith closed 6 years ago

MattKleinsmith commented 6 years ago

Warning: Click the top right point first, if using the current version of the repo.

Solution 1: Swap point_1 with point_2 if point_1 is the bottom right point. This solution will help with the VOC danger mentioned here: https://github.com/Cartucho/yolo-boundingbox-labeler-GUI/pull/6

Solution 2: Modify get_close_icon and draw_close_icon, and any other related functions

Cartucho commented 6 years ago

@MattKleinsmith I noticed this yesterday! My idea to solve is:

Selecting the bbox would also be great since we could later add an option to quickly resize and re-label the bounding box.

Cartucho commented 6 years ago

@MattKleinsmith It's done, now we can only delete if we select that bbox, it also deals with multiple bboxes goes for the smaller one: screenshot from 2018-02-16 16-39-23

I'm not sure if I thought of all the possible cases, but it is surely more robust now.