search

DynamicGravitySystems / DGP

Dynamic Gravity Processor
http://dgp.readthedocs.io/en/develop/
Apache License 2.0
7 stars 4 forks source link

Fix plotter auto-scaling issues. #57

Closed bradyzp closed 6 years ago

bradyzp commented 6 years ago

PR #56 introduces some changes to the plotter class which introduce some cases where scaling is not working correctly when adding a data series (sometimes resolved by clicking the plot home button).

This is introduced I believe by some code in plotter.py lines 984:992 where the margins are either statically computed, or in the case of the Y-axis, incorrectly.

This is partially addressed in the prototype StackedAxesManager class (dgp.gui.mplutils) introduced in PR #56 , and a hotfix will be made to the existing plotter code until the new API is complete.

bradyzp commented 6 years ago

Closing as this issue is no longer relevant, matplotlib has been deprecated in favor of PyQtGraph for interactive plots, auto-scaling is not an issue with the current implementation.