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

NameError: global name 'iterable' is not defined #1593

Closed
firebirdberlin opened this issue Dec 13, 2012 · 4 comments
Closed

NameError: global name 'iterable' is not defined #1593

firebirdberlin opened this issue Dec 13, 2012 · 4 comments
Milestone

Comments

@firebirdberlin
Copy link

I'm getting the following error:

Traceback (most recent call last):
File "./read_hdr.py", line 560, in
plot_sensor_locations(subdir);
File "./read_hdr.py", line 218, in plot_sensor_locations
ax.set_zlim(zlim);
File "/home/stefan/local_x86_64/Python/lib/python3.3/site-packages/mpl_toolkits/mplot3d/axes3d.py", line 680, in set_zlim3d
if top is None and iterable(bottom):

I think line 680 must read:

if top is None and cbook.iterable(bottom):

instead of

if top is None and iterable(bottom):

bye, Stefan :)

@pelson
Copy link
Member

pelson commented Dec 13, 2012

Thanks @firebirdberlin - thanks for suggesting the fix. Are you willing to give submitting a pull request a try? It would be a good experience to see how we make matplotlib code changes. If not, don't worry about it, I'm sure somebody will pick the change up at some point.

Many Thanks,

@firebirdberlin
Copy link
Author

I will give it a try, but firstly I'll have to escape my firewalled working place .... https pushing seems not to work ...

@WeatherGod
Copy link
Member

Looks like it was a bug I introduced recently when I copied and pasted code from the 2d axes. set_xlim3d(), set_ylim3d() and set_zlim3d() all have this bug. Also in set_zbound().

@WeatherGod
Copy link
Member

@firebirdberlin, thanks for catching this! Please make sure your PR is made against branch v1.2.x so that it will be fixed in both the maintenance branch and the master branch. Thanks!

WeatherGod added a commit to WeatherGod/matplotlib that referenced this issue Dec 20, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants