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

Heatmap with a large number of small clusters "flickers" when the map position is changed #9952

Closed
moneyatbo opened this issue Aug 25, 2020 · 3 comments
Assignees
Labels

Comments

@moneyatbo
Copy link

mapbox-gl-js version: 1.12.0

browser: Chrome 84.0.4147.135 (Official Build) (64-bit) (Mac, Linux, Windows)
Edge 84.0.522.63 (Official build) (64-bit)
IE 11.450
Safari Version 13.1.2 (15609.3.5.1.3)

Steps to Trigger Behavior

  1. Load page in the jsbin
  2. see the heatmap rendered over San Francisco
  3. with your mouse click and hold on the map
  4. now move the mouse to the left, right, up or down
  5. Notice that the heatmap does not render in the same locations as you pan.

Link to Demonstration

https://jsbin.com/zodewehiwa/edit?html,output

flicker

Expected Behavior

Consistent position of the heatmap clusters when panning over the map

Actual Behavior

Heatmap clusters migrate on the map as you pan.

I converted the Data Driven Circles example to be a heatmap to illustrate the problem we are seeing on our maps. https://docs.mapbox.com/mapbox-gl-js/example/data-driven-circle-colors/

@mourner mourner self-assigned this Aug 25, 2020
@mourner
Copy link
Member

mourner commented Aug 25, 2020

Yeah, duplicate of #5563. Heatmaps are rendered through a 16x downscaled framebuffer for performance, which probably causes the flickering due to lost information, but there may be a way to fix this without compromising performance — needs investigation.

@mourner mourner closed this as completed Aug 25, 2020
@moneyatbo
Copy link
Author

@mourner How easy is it to adjust the framebuffer if we forked the lib?

@mourner
Copy link
Member

mourner commented Aug 25, 2020

@moneyatbo I don't know. But if you do find a fix, we'll welcome a pull request!

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

2 participants