glue-viz / glue

Linked Data Visualizations Across Multiple Files
http://glueviz.org
Other
740 stars 153 forks source link

Fix issue where matplotlib viewers using ``y`` log axes were not initialized correctly. #2324

Closed Carifio24 closed 2 years ago

Carifio24 commented 2 years ago

This PR aims to fix #2194, which actually affects any of the matplotlib viewers that use log axes. The reason for the incorrect aspect described in the issue is that we currently only update the x log axes when setting up callbacks. This PR adds a call to update_y_log as well.

astrofrog commented 2 years ago

@Carifio24 - could you rebase to resolve the conflict in CHANGES.md?

Carifio24 commented 2 years ago

@astrofrog Should be good now!