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

run 2to3 for splot #6

Merged
merged 18 commits into from
May 15, 2018
Merged

run 2to3 for splot #6

merged 18 commits into from
May 15, 2018

Conversation

slumnitz
Copy link
Member

mapping.py can now be used in Python 3

only the last commit is new and I need it to use splot locally

jsignell and others added 18 commits May 8, 2018 17:12
(cherry picked from commit 545e9d3)
also removed plt.show call
added __all__ dict

(cherry picked from commit 1afe78b619a9424f845a28fbd5ee414637dff1d1)
also fixed type descriptors,
and changed fuction to return figure instance,
and made docstring conform to numpydoc standard

(cherry picked from commit f546b91065e95e935acc9488f24f344b567c01d5)
(cherry picked from commit ed130d4955624b83c836b66743169e96fbef34f3)
(cherry picked from commit f6b4e9752491986f8645572faae22d85efa2036f)
also made python mapping.py show both figures

(cherry picked from commit 385c835f6a52f5d9c7f6e2a89e4223868ad1b011)
(cherry picked from commit 6a8c7cd7bf1a144a2bbe6069ce87fe7df7efee88)
(cherry picked from commit f2456602c24ea8243e6d952d06fcdfadc470e700)
tests were failing now that libpysal no longer supports Python 2
See Seaborn pull request 97 for details
mapping.py can be imported now in Python 3
Copy link
Member

@sjsrey sjsrey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

Moving forward, python 3 only!

@sjsrey sjsrey merged commit 4435519 into pysal:master May 15, 2018
@ljwolf
Copy link
Member

ljwolf commented May 15, 2018

🎉

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.

6 participants