Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Geo projections getting clobbered by 2to3 when used when python3#1555

Merged
dmcdougall merged 1 commit intomatplotlib:masterfrom
joferkington:fix-geo-projections-py3k
Dec 4, 2012
Merged

Geo projections getting clobbered by 2to3 when used when python3#1555
dmcdougall merged 1 commit intomatplotlib:masterfrom
joferkington:fix-geo-projections-py3k

Conversation

@joferkington
Copy link
Copy Markdown
Contributor

projections/geo.py uses long as a variable name in many places. This is replaced with int by 2to3, which leads to problems (e.g. foo = int * bar instead of foo = long * bar).

Steps to reproduce:

  1. Install matplotlib with python3.x
  2. Run examples/api/custom_projection_example.py
  3. Mouse over the plot (or anything else that will call an inverse geo projection)

@efiring
Copy link
Copy Markdown
Member

efiring commented Dec 4, 2012

Looks good to me. "lon" and "lat" is much nicer than "long" and "lat" anyway.

@dmcdougall
Copy link
Copy Markdown
Member

Woah. Nice catch, @joferkington! This PR looks fine to me too. I'll merge this tomorrow. Thanks again.

@pelson
Copy link
Copy Markdown
Member

pelson commented Dec 4, 2012

👍

@mdboom
Copy link
Copy Markdown
Member

mdboom commented Dec 4, 2012

Thanks. This should be cherry-picked for v1.2.x.

@dmcdougall
Copy link
Copy Markdown
Member

Merging.

@mdboom Sure thing.

dmcdougall added a commit that referenced this pull request Dec 4, 2012
Geo projections getting clobbered by 2to3 when used when python3
@dmcdougall dmcdougall merged commit e5b0042 into matplotlib:master Dec 4, 2012
@dmcdougall
Copy link
Copy Markdown
Member

Cherry-pickled: 40d28e0

@joferkington
Copy link
Copy Markdown
Contributor Author

Thanks a bunch!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants