search

jveitchmichaelis / deeplabel

A cross-platform desktop image annotation tool for machine learning
207 stars 40 forks source link

Resize/move existing/detected annotations #29

Closed lukaswozniak closed 3 years ago

lukaswozniak commented 3 years ago

Is there any way to more/resize annotation? Currently the only way I found working is to remove annotation and add it again with corrected position and size. After running detector I find that many annotations are slightly inaccurate, and having easy way of correcting them would be a life saver.

jveitchmichaelis commented 3 years ago

@lukaswozniak Tentatively added in https://github.com/jveitchmichaelis/deeplabel/commit/156092c777445a97696547dbb8f1469368cfeae7

image

Give it a go! Hit the "select" icon and it should be obvious.

lukaswozniak commented 3 years ago

Awesome work man! Having a blast with it so far

jveitchmichaelis commented 3 years ago

Cool, let me know if you find any bugs. The only thing I thought of immediately was preventing dragging boxes beyond the extent of the window, but the bounding box values should be clamped on export anyway.

lukaswozniak commented 3 years ago

Well dragging the box out of the window is not something any user would want to do :) The only thing I've noticed so far is that if the boxes are overlapping then not always the selected one is prioritised.

lukaswozniak commented 3 years ago

For example: image

Here I'm trying to resize the smaller one, but the program is all confused and moves the bigger one, even though it colored the smaller corner

jveitchmichaelis commented 3 years ago

Hmm overlapping boxes is an awkward, but common, case.

I think one easy fix would be to add a Z-Order to each box and then give you the ability to adjust how the boxes stack on top of each other. I can also add a check so that if you select something like this, it will at least be non-ambiguous which box you're editing (e.g. once we determine we're in the hitbox of one of the labels, we should ignore the rest).

On Sun, Oct 3, 2021 at 4:39 PM lukaswozniak @.***> wrote:

For example: [image: image] https://user-images.githubusercontent.com/43893492/135763256-27b83e28-bb89-4122-995b-f1187d965161.png

Here I'm trying to resize the smaller one, but the program is all confused and moves the bigger one, even though it colored the smaller corner

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/jveitchmichaelis/deeplabel/issues/29#issuecomment-932984957, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAYDMJY3KMOMQAWZTXKWYPTUFCBN5ANCNFSM5FGBIJ2Q .

lukaswozniak commented 3 years ago

The best user experience I can imagine would be to mouse scroll between overlapping annotations.

lukaswozniak commented 3 years ago

Also btw, I feel the changes from last commits was pretty significant. Could you make new release? For most users compiling the project with all those dependencies is a no go.

jveitchmichaelis commented 3 years ago

Yep, I'll push something later today.

On Sun, Oct 3, 2021 at 5:22 PM lukaswozniak @.***> wrote:

Also btw, I feel the changes from last commits was pretty significant. Could you make new release? For most users compiling the project with all those dependencies is a no go.

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/jveitchmichaelis/deeplabel/issues/29#issuecomment-932992063, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAYDMJ5JAVL5PPJJIYJCX2DUFCGNFANCNFSM5FGBIJ2Q .