Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

[Android] Markers blink on zoom change #6762

Closed
mpuchala opened this issue Oct 19, 2016 · 18 comments
Closed

[Android] Markers blink on zoom change #6762

mpuchala opened this issue Oct 19, 2016 · 18 comments
Labels
Android Mapbox Maps SDK for Android

Comments

@mpuchala
Copy link

Platform: Android
Mapbox SDK version: 4.2.0 Snapshot

Steps to trigger behavior

  1. Create Map
  2. Zoom in/out

Actual behavior

It is also about the map. When I'm zooming in/out map and markers sometimes blink. It wasn't a problem on 4.1.0 now it occurs even on simple map. I've noticed similar behavior when I'm updating icon (in only one marker).

Here is a video:
https://www.dropbox.com/s/ok27ba09nty0sha/map_blink.mp4?dl=0

@tobrun tobrun added the Android Mapbox Maps SDK for Android label Oct 20, 2016
@mpuchala
Copy link
Author

mpuchala commented Nov 8, 2016

@tobrun

Hey,

Is there any progress with this issue?

I've just used 4.2.0-beta4 and it still appears. With few Markers UX is really bad (when I'm showing 10-20 almost everything on the screen is blinking). Are you able to reproduce it easily?

@cammace
Copy link
Contributor

cammace commented Nov 10, 2016

Hey @mpuchala, could you clarify whether you are using GL Markers or view based markers? Also does this issue occur when you are using just the default markers found in the SDK?

@mpuchala
Copy link
Author

mpuchala commented Nov 12, 2016

Hey,

Here are examples from Dev Preview:

https://www.dropbox.com/s/ptckispqoc077np/map_blink_2.mp4?dl=0
https://www.dropbox.com/s/ko0u1wkd3qfpgl3/map_blink_3.mp4?dl=0

I'm using GL Markers, there is a default icon and it's easy to find out same problem. It seems to blink when other elements like labels, icons, etc. are changing. As I mentioned before it also occurs when I'm changing an icon (all markers are disappearing for a moment) - it isn't an issue on 4.1.0 SDK version.

@cammace
Copy link
Contributor

cammace commented Nov 16, 2016

You can try switching to marker views which shouldn't have any flickering issue. I've been able to reproduce in the test app with many markers:

ezgif com-video-to-gif 4

Any idea what is happening here @ivovandongen?

@mpuchala
Copy link
Author

Hey,

Thanks for reply.

My problem with MarkerViews is performance. I tried to use ~200 of them and app almost haven't responded.

IMHO it's not only about amount (GL Markers) but for sure it's easier to notice this behaviour then - in example I've attached there is only one icon.

Markers seem to disappear for a while when other map components like labels are being changed (and also only one marker icon I mentioned above).

We're using at least 1000-2000 markers so probably MarkerViews aren't suitable in our case - I've tested them with only one icon and result wasn't promising. I use them (MV) when I have few points and it works like a charm in that cases.

@tobrun
Copy link
Member

tobrun commented Dec 13, 2016

Have seen multiple reports on this, going to try pinpointing the cause of this.

@tobrun
Copy link
Member

tobrun commented Dec 14, 2016

a git bisects indicates this is coming from 41bbd4e. I'm thinking it will be related to the geometry_tile_worker? Any tips on how to proceed on this @jfirebaugh?

@jfirebaugh
Copy link
Contributor

Do you have a test case?

@tobrun
Copy link
Member

tobrun commented Dec 14, 2016

It's reproducible with the Bulk Marker Activity from the testapp as shown by @cammace in #6762 (comment). I think it's hard to write a test for this since it happens as part of gesture.

@dylan-chong
Copy link

Has any progress been made on this issue in the last 3 months? This is a pretty problematic bug in terms of user experience, so I can't actually use any 4.2.x version on my app.

@tobrun tobrun removed their assignment Apr 19, 2017
@tobrun tobrun added this to the android-v5.1.0 milestone Apr 20, 2017
@tobrun
Copy link
Member

tobrun commented Apr 20, 2017

Going to add this to next milestone to look into more.

@mpuchala
Copy link
Author

@tobrun
Have no idea if this feedback would be helpful or not, but on 5.0.2 it seems to work fine and stable on my app (~200 Markers - no MarkerViews), unfortunately it's an issue again on 5.1.0 beta...

@mpuchala
Copy link
Author

mpuchala commented May 18, 2017

@PhanVanLinh If you would like to find differences please take a look on examples and API:
https://www.mapbox.com/android-sdk/api/5.0.2/

MarkerViews are much more flexible, however you have to prepare adapter for them (there is no need like that for Markers). According to my experience Markers are much more faster than MarkerViews (at least there were when I was compering them some time ago).

I'm not a MapBox team member so maybe sth has changed and I haven't noticed, but I think it's not the best place to discuss about them - it's off topic and this issue is about something else.

@PhanVanLinh
Copy link

PhanVanLinh commented May 18, 2017

@mpuchala Thank you so much for your explain. I will read the API. if you don't mind. can you share me your email then we can discuss in another place. my email is: phanvanlinh.94vn@gmail.com. I will remove this comment after your response. I saved your answer and if you want you can delete your comment. Again thank you so much

@tobrun tobrun modified the milestones: android-v5.2.0, android-v5.1.0 Jun 12, 2017
@lilykaiser lilykaiser modified the milestones: android-v5.2.0, android-v6.0.0 Sep 25, 2017
@paulVulog
Copy link

Just in addition, I am able to see some flickering by just playing with the zoom in the MapBox demo app (v5.1.3) in the Annotations/Draw a marker Activity.
It happens at very specific zoom level (e.g : at map labels woo; changes)

@VGordejevs
Copy link

Yeah, weird story :x (5.2.0-SNAPSHOT)

@fdermishin-radargeo
Copy link

5.2.0 still has this issue. This happens when layers change while zooming. For some reason there is a frame, on which new layers are already drawn, but layer with markers is not visible. The markers appear only on the next frame.

@tobrun
Copy link
Member

tobrun commented Dec 5, 2017

This behaviour is fixed on master and is planned to be released as part of the upcoming 6.0.0 release.

@tobrun tobrun closed this as completed Dec 5, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Android Mapbox Maps SDK for Android
Projects
None yet
Development

No branches or pull requests