From: Eric F. <ef...@ha...> - 2007-09-09 23:28:50
|
The Axes.panx() method and several others contain calls to _send_xlim_event() and similar methods. These methods don't seem to be defined anywhere. Eric |
From: John H. <jd...@gm...> - 2007-09-10 00:58:50
|
On 9/9/07, Eric Firing <ef...@ha...> wrote: > The Axes.panx() method and several others contain calls to > _send_xlim_event() and similar methods. These methods don't seem to be > defined anywhere. These methods (panx, zoomx) are from the very old toolbar, which should be deprecated and removed in my option. It doesn't look like they work anyhow, because as you note they rely on the nonexistant _send_xlim_event. The xlim callbacks are handled by the 'xlim_changed' notification of the callbacks attribute. I think all the methods that contain this (panx, zoomx and PolarAxes.set_xlim can either be removed entirely or have the _send_xlim_event call removed. JDH |
From: Eric F. <ef...@ha...> - 2007-09-10 01:48:07
|
John Hunter wrote: > On 9/9/07, Eric Firing <ef...@ha...> wrote: >> The Axes.panx() method and several others contain calls to >> _send_xlim_event() and similar methods. These methods don't seem to be >> defined anywhere. > > These methods (panx, zoomx) are from the very old toolbar, which > should be deprecated and removed in my option. It doesn't look like > they work anyhow, because as you note they rely on the nonexistant > _send_xlim_event. The xlim callbacks are handled by the > 'xlim_changed' notification of the callbacks attribute. I think all > the methods that contain this (panx, zoomx and PolarAxes.set_xlim can > either be removed entirely or have the _send_xlim_event call removed. > > JDH John, OK, thanks, I will try to clean this up soon. I stumbled over it because one of the wx examples has its own toolbar with buttons that call these methods. Eric |
From: Eric F. <ef...@ha...> - 2007-09-10 06:43:33
|
John Hunter wrote: > On 9/9/07, Eric Firing <ef...@ha...> wrote: >> The Axes.panx() method and several others contain calls to >> _send_xlim_event() and similar methods. These methods don't seem to be >> defined anywhere. > > These methods (panx, zoomx) are from the very old toolbar, which > should be deprecated and removed in my option. It doesn't look like > they work anyhow, because as you note they rely on the nonexistant > _send_xlim_event. The xlim callbacks are handled by the > 'xlim_changed' notification of the callbacks attribute. I think all > the methods that contain this (panx, zoomx and PolarAxes.set_xlim can > either be removed entirely or have the _send_xlim_event call removed. These methods have been broken for a long time, which is a crude form of deprecation. I am simply removing them now. Same for the old toolbar. Given that the pan and zoom functions have been broken for so long, I don't see how anyone could be using it, so I don't see any point in keeping it around with a deprecation warning. Eric > > JDH |