search

glue-viz / glue-plotly

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

Add unit tests for 2D scatter exporter #36

Closed Carifio24 closed 1 year ago

Carifio24 commented 1 year ago

This PR adds some unit tests for the 2D scatter exporter, as well as fixes some issues that were found during the creation of these tests. The tests here are relatively straightforward and confirm that the generated Plotly traces have the expected properties. More PRs to come with similar tests for the other viewer types.

Carifio24 commented 1 year ago

Looks like the issue we're seeing on macOS is https://github.com/actions/setup-python/issues/682

Carifio24 commented 1 year ago

In the interest of keeping things moving forward, I've temporarily replaced the Python 3.7 CI with Python 3.8 until the issue mentioned above has been resolved.