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

map.addLayer(..."circle-radius"..."stops": ...) throws error "stop domain values must be unique" #4218

Closed
datadiver0x0 opened this issue Feb 7, 2017 · 3 comments · Fixed by #10846

Comments

@datadiver0x0
Copy link

Context: data-driven styling for circle radius; zoom and property functionality broken

In the newest Mapbox-gl release, v0.32.1, data-driven styling for circle radius no longer works as described in the Mapbox GL data-driven style reference's "Circle layer, zoom and circle-radius, exponential" example.

Setting combined zoom/property value stops using the [{ "zoom": 15, "value": 1 }, 10] format no longer works if the same value is repeated for a different zoom level (e.g. [{ "zoom": 17, "value": 1 }, 20] ). This syntax produces the error "stop domain values must be unique". This may be the result of an edit based on issue #611, "Require unique categorical function domain values".

A minimally altered version of the data-driven styling example can be found here. I had to alter it because the original example didn't perform any data-driven styling on the circle radius. Reverting to version v0.31.0 results in the expected functionality.

As a side note, links to "zoom and property functions docs" appear to be broken.

Thanks,
Jus

@jfirebaugh
Copy link
Contributor

Thanks for the report @datadiver0x0. This was indeed an unintended result of mapbox/mapbox-gl-style-spec#611. Fix in #4220.

@datadiver0x0
Copy link
Author

datadiver0x0 commented Feb 8, 2017 via email

@lucaswoj
Copy link
Contributor

@datadiver0x0 Our next release will be on or before March 1st

asheemmamoowala pushed a commit that referenced this issue Jul 14, 2021
* Update style spec compatiblity for #9847 and #4218

Co-authored-by: Karim Naaji <karim.naaji@gmail.com>
@asheemmamoowala asheemmamoowala linked a pull request Jul 14, 2021 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants