Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
VyacheslavLukin opened this issue Jul 26, 2017 · 8 comments
Open
Labels

Comments

@VyacheslavLukin
Copy link

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

@AlNedorezov
Copy link
Collaborator

AlNedorezov commented Jul 26, 2017

@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:

  • the version of the system you are using,
  • was the issue spotted during the acceptance of the markup, or during the markup process itself,
  • are there any error messages in the chrome console after zooming images,
  • used OS,
  • used browser and its version.

@AleksandrShepelev
Copy link

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
Copy link
Author

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
Copy link
Author

VyacheslavLukin commented Jul 27, 2017

To answer your questions:

  • Ubuntu Linux 16.10
  • During acceptance
  • there is the error message
    image
  • Google Chrome | 59.0.3071.115 (Official Build) (64-bit)
    Revision | 3cf8514bb1239453fd15ff1f7efee389ac9df8ba-refs/branch-heads/3071@{#820}
    JavaScript | V8 5.9.211.38

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
Copy link
Collaborator

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

@AleksandrShepelev
Copy link

AleksandrShepelev commented Sep 11, 2017

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
Copy link

AleksandrShepelev commented Sep 11, 2017

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

  2. increase zoom in app:
    image

  3. 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
Copy link

AleksandrShepelev commented Sep 11, 2017

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants