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

Fix landcover flicker in Mapbox Streets style #1875

Closed
averas opened this issue Dec 26, 2015 · 17 comments
Closed

Fix landcover flicker in Mapbox Streets style #1875

averas opened this issue Dec 26, 2015 · 17 comments

Comments

@averas
Copy link
Contributor

averas commented Dec 26, 2015

Perhaps this is a dupe, but I didn't find any other issue on it..

When zooming in/out on streets what I believe is the landcover layer starts to flicker considerably. This does not seem to appear on the other styles, so it seems it is something related to this particular layer in the streets style itself.

flicker2

@lucaswoj
Copy link
Contributor

lucaswoj commented Jan 4, 2016

Not a dupe, as far as I know, but it reminds me a little of #1270. Can you repro this across all browsers?

@averas
Copy link
Contributor Author

averas commented Jan 4, 2016

My report was made on Chrome/OSX. Same experience in Safari/OSX and Firefox/OSX. Tested on a Mac with retina resolution.

I don't have easy access to IE.

@ansis
Copy link
Contributor

ansis commented Jan 5, 2016

002c690 fixes this

@ansis ansis closed this as completed Jan 5, 2016
@averas
Copy link
Contributor Author

averas commented Feb 4, 2016

Was this actually fixed? Or is it back? I see considerable flicker on streets when zooming slowly in and out around zoom level 11-13.

@ansis ansis reopened this Feb 4, 2016
@ansis
Copy link
Contributor

ansis commented Feb 4, 2016

I'm not able to reproduce this anymore on master. Does it still look like the flickering in the original gif, or is it different now?

@averas
Copy link
Contributor Author

averas commented Feb 4, 2016

Well.. it might be a little bit more subtle.. but I don't know. Fresh video from Mapbox GL JS Examples (which I believe is using 0.13.1):

new_flicker

My impression is that the effect is more clear in Mapbox Studio, either just perceptually because the map canvas is larger, or because rendering is a bit slower.

OS.X (El Capitan), Chrome, Retina Display.

@lucaswoj lucaswoj changed the title Flicker on streets Fix landcover flicker in Mapbox Streets style Jul 29, 2016
@lucaswoj lucaswoj removed the bug 🐞 label Aug 4, 2016
@mourner
Copy link
Member

mourner commented Apr 19, 2017

@stevage do you still see this issue? I can't reproduce this on latest stable Chrome / OS X / Retina.

@stevage
Copy link
Contributor

stevage commented Apr 19, 2017

Me? Don't think I ever saw it. But for the record, I don't seem to be able to reproduce this effect using Mapbox Streets now.

@averas
Copy link
Contributor Author

averas commented Apr 19, 2017

I think that the Mapbox core styles have been updated since I originally filed this issue making it a bit harder to reproduce, but I was still able to. To make it clearer I made a custom style based on Mapbox streets - nothing added, just removed a bunch of layers:

Goto https://api.mapbox.com/styles/v1/averas/cj1pk7b7d005q2so1r3t5kiag.html?title=true&access_token=pk.eyJ1IjoiYXZlcmFzIiwiYSI6ImRrb1Z6NXcifQ.8v3wimUeRVTvOMbozrwkfw#2.7/-2.02/27.21

Start zooming in slowly over Africa. I see flickering starting around zoom 3 and intensifying up until ~ zoom 6 and it diminishes at zoom 7. It's clearly related to the opacity ramps of the grass/crop landcover layers. My impression is that the opacity ramp is non-linear perceptually, as it gives a subtle impression that the color oscillates between greenish/brownish.

flicker-new

MacBook Pro (Retina, 15-inch, Late 2013) / Chrome Version 57.0.2987.133 (64-bit) (Also tested in Safari and Firefox)

@jfirebaugh
Copy link
Contributor

Very strange. I see what you're talking about in the gif, but I can't reproduce that effect locally either. MacBook Pro (Retina, 13-inch, Early 2015) / Chrome 57.0.2987.133.

@stevage
Copy link
Contributor

stevage commented Apr 19, 2017

Me too - see it in the gif, not for real though. Macbook Pro Retina 15 inch mid-2014 (wow, is it that old already?), Chrome 57.0.2987.133 (and Firefox) either on laptop screen or externally. Also tried using either touchpad or scroll mouse.

@averas
Copy link
Contributor Author

averas commented Apr 19, 2017

@jfirebaugh / @stevage Yes, that's really strange. Perhaps model-specific then.. I will check with some newer MacBooks at the office tomorrow.

I have the NVIDIA GeForce GT 750M (2048 MB) + Intel Iris Pro 5200 (1536 MB), but that ought to be the same configuration as @stevage has...

@stevage
Copy link
Contributor

stevage commented Apr 20, 2017

Yep, "Graphics Intel Iris Pro 1536 MB".

I vaguely recall changing some WebGL settings in Chrome once...but you say you see it in Firefox too.

Are you able to produce an example with a different dataset? Or at least, style it so the difference in colours is clearer?

@mourner
Copy link
Member

mourner commented Apr 20, 2017

I can finally see the flickering with the test case above (although it's quite subtle). MBP Retina mid-2012, Nvidia GT 650M, Chrome.

@jfirebaugh
Copy link
Contributor

Is this still reproducible? It may have been the same issue as #5490 / #5704 / https://bugs.chromium.org/p/chromium/issues/detail?id=784030.

@averas
Copy link
Contributor Author

averas commented Jun 5, 2018

It's still reproducible for me on Chrome/OSX, Safari/OSX and Firefox/OSX, on my very same good old MacBook Pro (Retina, 15-inch, Late 2013) with NVIDIA GeForce GT 750M (2048 MB) + Intel Iris Pro 5200 (1536 MB) using the same map as above:

https://api.mapbox.com/styles/v1/averas/cj1pk7b7d005q2so1r3t5kiag.html?title=true&access_token=pk.eyJ1IjoiYXZlcmFzIiwiYSI6ImRrb1Z6NXcifQ.8v3wimUeRVTvOMbozrwkfw#2.7/-2.02/27.21

I suspect it's platform/hardware related though.. trying out the same map on a newer Mac did not render the same issue.. so perhaps it's not worth spending time on this.

@mourner
Copy link
Member

mourner commented Aug 1, 2018

I suspect it's platform/hardware related though.. trying out the same map on a newer Mac did not render the same issue.. so perhaps it's not worth spending time on this.

Yeah, it seems like there's nothing we can do on the GL JS side to address this, and given low severity of the issue, I think it's safe to close.

@mourner mourner closed this as completed Aug 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants