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

Python3 compatibility #7

Open
sina-mansour opened this issue Oct 24, 2018 · 7 comments
Open

Python3 compatibility #7

sina-mansour opened this issue Oct 24, 2018 · 7 comments

Comments

@sina-mansour
Copy link

I was wondering if it's possible to make the codes python3 compatible?

@unidesigner
Copy link
Contributor

Not planned, but happy to receive a pull request.

@sanjayankur31
Copy link

Python2 will be EOL soon, and most libraries have therefore moved to Py3. I could do a 2to3 conversion and open a PR, but I don't know enough about the library to do a complete py3 conversion---someone from the development team will have to do that bit.

@unidesigner
Copy link
Contributor

Hi! Thanks for the pull request. Are you using the library?

@sanjayankur31
Copy link

Unfortuntely, I do not use cfflib in my work. We're looking at including cfflib in NeuroFedora since it's commonly used by lots of other tools. We do not support py2 only libraries, though, so it'll be nice to have cfflib updated to py3.

https://pagure.io/neuro-sig/NeuroFedora/issue/73

https://docs.fedoraproject.org/en-US/neurofedora/overview/

@unidesigner
Copy link
Contributor

Can you name the tools that use it? It was used by connectomeviewer, but not sure if many people use it in fact. Any others? It might be better to stop supporting the packages for the NeuroFedora community (or the NeuroDebian community). As you can see from the commit log, the package is pretty much not maintained.

@sanjayankur31
Copy link

Here's one that I've come across:

https://github.com/the-virtual-brain/tvb-data/blob/7d2d05b805aeed0b4d3fe26d2ba97f13341ce65a/tvb_data/cff/networkx_migration.py

I agree. If the package is unmaintained, we will not include it. If cfflib is not maintained and will not be updated in the future either, can you please state this clearly in the README and suggest an alternative if there is one? That'll enable others to request whatever tools use cfflib to stop doing so.

@unidesigner
Copy link
Contributor

It is so old, I do not even have write access to the repo anymore to update the README. I'll need to see if any of the admins is still responsive to add the note in the README. I'd suggest to remove it, it won't be maintained or updated. Something that might be interesting for people who were interested in cfflib is BIDS http://bids.neuroimaging.io/

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