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

Sugget to add python2.7 to documentation #25

Open
qiyubio opened this issue Dec 23, 2020 · 5 comments
Open

Sugget to add python2.7 to documentation #25

qiyubio opened this issue Dec 23, 2020 · 5 comments

Comments

@qiyubio
Copy link

qiyubio commented Dec 23, 2020

Hi,

We found that tadtool installed using python3 will give error “colorbar” had no attribute “set_clim”.
This error is from matplotlib3, could be solved by installing tadtool in python2.7 environment.
So to avoid similar problem in other user, I suggest to add something like 'tadtool is a Python 2 compatibility package' in README.

Best,
Qi

@rayoq2021
Copy link

Hello Qi,
Can you post the steps you performed to enable TADtool to run on python2.7? I am very much interested in running TADtool and would be greatfull to see the steps and the installation of the other required packages like Numpy, Matplotlib, etc.

Thanks in advance for your assistance,
Ray

@kaukrise
Copy link
Collaborator

kaukrise commented Jan 5, 2021

Hey guys,

sorry, I'm still technically in holiday, and I'll attempt to fix this properly once I return. For now, downgrading matplotlib should also work, maybe you can give that a try first?

pip install matplotlib==3.2.2

@kaukrise
Copy link
Collaborator

kaukrise commented Jan 5, 2021

Never mind, the fix was not all that complicated, it seems. I pushed a new version to Pypi (0.8.2) which restores matplotlib 3.3 compatibility and it works fine in my tests in Python 3. There is a nasty deprecation warning though, which I will have to attend to at some point, but everything seems to work as expected. Please let me know if you run into any additional issues!

@rayoq2021
Copy link

rayoq2021 commented Jan 7, 2021 via email

@kaukrise
Copy link
Collaborator

kaukrise commented Jan 8, 2021

Hey, you need the latest version of tadtool with the fix I implemented.

pip install --upgrade tadtool

PS: If possible, please reply in GitHub and not via email. The formatting in your last post was all screwed up and there was a lot of nonsensical text in there which I had to remove before even finding your message. Thanks!

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

No branches or pull requests

3 participants