search

innosoft-pro / label-them

MIT License
6 stars 3 forks source link

Selection messed up with labelling #19

Open nmingazov opened 7 years ago

nmingazov commented 7 years ago

Screencast in gif on pretty random hosting :) https://ezgif.com/optimize/ezgif-3-9d4bcadb3e.gif

AlNedorezov commented 7 years ago

If we move select polygon (labelled object) action from "hand" tool to starting markup with "polygon" tool from inside already marked up object, then the system will not fully support use cases in which one smaller object covers an internal part of the bigger object (e.g., if on the image below we have already marked up the rectangular object, we won't be able to markup the circle object) Example of a use case in which one smaller object covers an internal part of the bigger object

mihbort commented 7 years ago

It is better to make it not possible to create new blob, while class of the previous one is not marked. It would decrease human mistakes during the marking.

AlNedorezov commented 7 years ago

@mihbort, thank you for your feedback. We'll take it into account.

AlNedorezov commented 7 years ago

Since this issue is related to the question of usability of distribution of markup functionality into two different tools, and as starting from release 1.7 functionality of "Polygon" and "Hand" tools are unified into a singe tool, I think that that issue can be closed.