search

innosoft-pro / label-them

MIT License
6 stars 3 forks source link

Poligons move instead of zoom (when they should zoom as well) #32

Open VyacheslavLukin opened 7 years ago

VyacheslavLukin commented 7 years ago

Issue presented on the attached screencast screencast-2017-07-26_17 07 23

AlNedorezov commented 7 years ago

@VyacheslavLukin, I was not able to reproduce the described issue. I am using release v1.5. Is it a common behavior, you are experiencing with zoom-in, zoom-out buttons? Please share the following information relating to the described issue:

AleksandrShepelev commented 7 years ago

We use version 1.5 Sometimes during the acceptance I noticed contours on wrong place (like on Vyacheslav GIF). But bug is not reproduced every time.

VyacheslavLukin commented 7 years ago

Moreover, sometimes we see pictures like following image As you can see this is very similar to the GIF. I doubt that person who marked picture, places polygons "in the air". However, we could not accept this picture, or can we? Maybe it's representation issue?

VyacheslavLukin commented 7 years ago

To answer your questions:

Information for the user from toloka: OS - windows 10 Browser - Chrome 59.0.3071

Another user for the same issue: OS - IOS 10.10.3 Browser - MOBILE_SAFARI 10.10.3

One more: Windows 10 Opera 46

AlNedorezov commented 7 years ago

@VyacheslavLukin, thank you! We will look into it.

AleksandrShepelev commented 7 years ago

I found how to reproduce it in chrome (windows):

  1. increase zoom by app tool. image

  2. mark a car

image

  1. decrease zoom in browser (not an app tool)
    image

polygons will move. Same happens in many combinations of browser zoom and your tool zoom.

AleksandrShepelev commented 7 years ago

in acceptance mode, there is a way to "fix" moved markups (but I don't know, if it can really fix it or not):

  1. Moved markup: image

    1. increase zoom in app: image

  2. decrease in browser (I've repeated cycle 2-3 two times to achieve result): image

can we fix it that way and accept or they will be of a bad quality?

AleksandrShepelev commented 7 years ago

Finally, we caught the bug.

  1. Create new pool.
  2. Markup one picture with one zoom level (default)
  3. Move to next picture
  4. Markup second picture with "+" zoom in app
  5. Move to next picture with zoomed current picture
  6. Markup third picture without zoom again. result: polygons ok - polygons moved - polygons moved

so, it probably means that zoom state is saved between pictures