-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Add support for custom geojson polygons in choropleth traces #4267
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
Comments
Our current topojson files (built in https://github.com/etpinard/sane-topojson using Natural Earth data) contain:
The "full" subunit lists are available here. In general, we can't use a single I suspect what you're really looking for is either:
|
@nicolaskruchten I suspect you're looking for us to implement
item here? |
Yup. TopoJSON optional for now but basically the same API as we did for mapbox would be great, plus some automatic bounding box logic |
Yes, that's what I'm looking for. The first option also would be nice, but actually it's not hard to make (or find) topojson files for different countries. Thanks for translating my description into a proper feature request. |
Here's my WIP branch: where a new Now regarding
This is essentially the I'm thinking of adding a new That's probably a couple more days of work; nothing substantial. Does that sound ok? |
... oops, I forgot to paste the link to the demo:https://codepen.io/etpinard/pen/zYxOoEB |
Yes! Can we default it to |
Also: this would be |
We could, but I'd vote against it for consistency with |
Yes, that's the plan for |
what would |
Having thought about this a bit, I think |
Sure, good call. |
Locationmode right now only accepts these 3 options: "ISO-3" | "USA-states" | "country names"
I want to make choropleth maps for different countries on provinces level, as equivalent to locationmode: "USA-states". Instead of defining a new locationmode for every country, like "BRA-states" or "ESP-states" maybe it would be better to define a new generic one calles "province" and afterwards define the country using locations option.
The text was updated successfully, but these errors were encountered: