plotly.py icon indicating copy to clipboard operation
plotly.py copied to clipboard

Published 20 hours ago • verified publisher icon plotly

Add new `map` options to `px` and update plotly.js/master at 056799dfc705a4533935ae3169cc94d35bc44830

Open archmoj opened this issue 1 year ago • 5 comments
trafficstars

Related to https://github.com/plotly/plotly.js/pull/7015.

archmoj avatar Aug 23 '24 16:08 archmoj

cc: birkskyum

archmoj avatar Aug 23 '24 18:08 archmoj

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

LiamConnors avatar Aug 27 '24 19:08 LiamConnors

Is it possible to add a deprecation warning for Mapbox traces similar to plotly/plotly.js#7087, but in this case have it appear in the Python console in addition to in the browser console?

If possible, that should be done in a separate PR. This PR is already huge.

archmoj avatar Aug 27 '24 20:08 archmoj

@archmoj makes sense to me -- opened #4730 to cover that case.

ndrezn avatar Aug 28 '24 14:08 ndrezn

@LiamConnors Please take over this PR and once ready merge it into your doc PR or master? I thought you may want to revert big commit of plotly.js update ba9c413 and just add the px parts to have viewable diffs. No?

archmoj avatar Aug 28 '24 17:08 archmoj