Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Undefined elements in axes module #1465

Closed
NelleV opened this issue Nov 8, 2012 · 3 comments · Fixed by #1569
Closed

Undefined elements in axes module #1465

NelleV opened this issue Nov 8, 2012 · 3 comments · Fixed by #1569

Comments

@NelleV
Copy link
Member

NelleV commented Nov 8, 2012

Dixit pyflakes, there are several undefined elements in the axes module:

  • The method Axes.__pick (line 3240) calls test, which isn't defined
  • The method Axes.dist (line 3221) calls Patch and Text which haven't been imported

Thanks,
N

@pelson
Copy link
Member

pelson commented Nov 8, 2012

I've labelled these as bugs. It may be that there are no code paths to trigger them, in which case, we can just remove the code.

Thanks for raising @NelleV .

@NelleV
Copy link
Member Author

NelleV commented Dec 7, 2012

I still don't know what to do here: I tend to think that as the code does not work since 2006, we should probably remove it (less code to maintain is always a good thing).

Maybe @efiring, being the author of this code, can comment on this.

@efiring
Copy link
Member

efiring commented Dec 7, 2012

Rip out the entire __pick() method. Everything down to the "### Labelling" comment. The sooner the better. I don't know how it survived so long--probably because axes.py is so huge that obsolete junk like this can hide in it for years.

I'm not actually the author of this; I come up with "blame" because of a fairly large reorganization of axes.py that I did long ago--basically shuffling things around so that I could find things.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants