glue-viz / glue-plotly

Experimental plot.ly plugin for glue
BSD 3-Clause "New" or "Revised" License
2 stars 8 forks source link

Refactor 3D scatter functionality #33

Closed Carifio24 closed 1 year ago

Carifio24 commented 1 year ago

This PR refactors the functionality for 3D scatter export to match what was done for 2D scatter, histogram, and profile export. This doesn't yet add the 3D scatter viewer to the web exporter - that can wait for a future PR. The motivation for this, and other refactors, is to make it easily to expose this functionality to other export contexts in the future (e.g. glupyter).

Additionally, this PR allows the 2D scatter hovertext array to be smaller by taking the size of the sanitizing mask into account.