-
Notifications
You must be signed in to change notification settings - Fork 21
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
new_axes=False + soma outline fix #30
Conversation
@wizmer , what is the best way to fix the python2 issue with latest matplotlib? They made some changes and it prints warnings if we create several axes with same name. It works fine for python3, but latest matplotlib is not python2. Do you guys want to drop python2? |
Yes, we want to drop python 2 :) |
ok, I let you sort this out then, not sure how to remove py27 checks, just from tox.ini? |
It needs to be a separate commit bumping the minor version. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
I'm too late but usually when making a change as big as deprecating a python version, it is good that it appears in the commit title. |
one for not adding new axes (matplotib warns that this may not work as it is in future versions), and use plain soma plot (the contour is a bit buggy with shifted neurons)