Line2D.set_pickradius missing in svn

Development matplotlib-devel
#1

Hi,

I just updated my checkout to rev 6829, and it seems lines.Line2D.set_pickradius has been renamed to setpickradius. Is this a typo? get_pickradius still exists. This is on line 318 in lines.py. Renaming it back to set_pickradius seems make it work the way it used to.

Cheers,

Martin

#2

Martin Spacek wrote:

Hi,

I just updated my checkout to rev 6829, and it seems lines.Line2D.set_pickradius has been renamed to setpickradius. Is this a typo? get_pickradius still exists. This is on line 318 in lines.py. Renaming it back to set_pickradius seems make it work the way it used to.

John, you made the change on Dec. 10:

http://currents.soest.hawaii.edu/hg/hgwebdir.cgi/matplotlib_mirror/diff/0a8f5203a8fd/matplotlib/lib/matplotlib/lines.py

Looks accidental to me.

Eric

#3

Yep, it must have been a bug because it doesn't follow the naming
conventions for the set_property funcs. Fixed on branch and trunk.

Thanks for tracking me down as the culprit :slight_smile:
JDH

ยทยทยท

On Tue, Jan 27, 2009 at 12:28 AM, Eric Firing <efiring@...229...> wrote:

Martin Spacek wrote:

Hi,

I just updated my checkout to rev 6829, and it seems
lines.Line2D.set_pickradius has been renamed to setpickradius. Is this a
typo? get_pickradius still exists. This is on line 318 in lines.py. Renaming
it back to set_pickradius seems make it work the way it used to.

John, you made the change on Dec. 10:

http://currents.soest.hawaii.edu/hg/hgwebdir.cgi/matplotlib_mirror/diff/0a8f5203a8fd/matplotlib/lib/matplotlib/lines.py

Looks accidental to me.