Liam Connors

Results 66 comments of Liam Connors

Yes, I believe it's this issue with plotly.py 6: https://github.com/plotly/plotly.py/issues/5119 The example works with 5.24

> Here is a list of doc files that has `mapbox` in their name: > > ``` > ./doc/python/mapbox-county-choropleth.md > ./doc/python/mapbox-layers.md > ./doc/python/filled-area-on-mapbox.md > ./doc/python/mapbox-density-heatmaps.md > ./doc/python/lines-on-mapbox.md > ./doc/python/scattermapbox.md >...

> Yes we could keep mapbox names in this PR. But for the other one we want to merge those should be renamed as we need to keep doc for...

> We should keep some mapbox examples, but like in `plotly.py` I would expect a deprecation notice associated with them. With the major version where we fully drop support for...

This looks good to me. Was there a particular example you couldn't get to render @archmoj

@gvwilson I don't think so. It doesn't sound like it's related to our docs, but rather to one of the Plotly.py renderers

The documentation update is still an issue. It's not an easy one to resolve as the the documentation is autogenerated from Plotly.js and probably requires some updates to Plotly.js.

Thanks @thihanmoekyaw for the PR! The one issue I can think of with adding them in this way is that they can get out of date when there are Dash...

@thihanmoekyaw let me know if you'd be interested in updating the PR to link out to the properties instead