-
Notifications
You must be signed in to change notification settings - Fork 90
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 support? #100
Comments
for what it's worth I was able to get poretools working in python3 with the following commands(standard linux):
after this I rebuilt in python2 and python3 and it seems to work fine. If there is interest I would be happy to place a pull request, else one of you could run the same. |
A poretools conversion to python 3 would be much appreciated pfaucon. I wouldn't be able to help to achieve this since I am fairly new to bioinformatics but I am happy to offer the computer systems that I have available for testing of any new builds. |
This is great, thank you for the suggestion. @arq5x what do you think? |
I am on board with this, but don't have any direct experience doing this. @tomsasani would you be up for tackling this with a bit of guidance from @brentp, as he has done 2 and 3 support for some of his tools. |
it's already done, look at the pull request that I opened in October. It is probable that there will be more changes to be made (or one could re-run my original pipeline to do the conversion). I think the main problem is that there are no unit tests for pore tools so I did limited testing by hand at the time, but it would be more comforting to have a standardized testing. |
@arq5x @nickloman @brentp @tomsasani It seems like poretools hasn't had any maintenance for at least a few months, but it is still an important tool to the Nanopore community. There are numerous new tools being released, and new file formats and folder formats being used. Is there any interest in continuing the development, or at least maintaining it? I would be willing to make some contributions, as I have used the tool, but only if they would actually be reviewed and possibly incorporated(unlike my proposed patch that has now sat for a few months :) ). |
Well I think between us we've managed to keep up with the various changes required for new Metrichor (recursive folder operation) and the new MinKNOW 1.4 file format, but we are certainly overdue rolling a new release. What we really could do productively is spend a couple of days looking through all the issues and pull requests and cleaning up the repository. |
Agreed. Let's do that and yes, contributions from others are most welcome. Sorry for the slowness in integrating. We are juggling many software projects and just don't have enough hands on deck! |
Cross referencing this pull request to add python3 support. |
I'm looking to rewrite some of the scripts we were using for fast5 parsing, and in particular to just borrow functionality from poretools but I'm having trouble running with python3. In particular pip install fails because setup.py has a "print version" instead of "print(version)" I'll submit a fix for that, but I wanted to see if I can get everything up to speed at the same time. In particular I'm seeing the error below about "scripts" missing, this error does not show up in python2.7.
It looks like there is no support for python 3 within the package (imports are relative, prints and exceptions lack parenthesis etc...) is there interest from the group in doing the conversion?
pfaucon@pfaucon-ubuntu:~/Desktop/poretools$ python3
Python 3.4.3 (default, Oct 14 2015, 20:28:29)
[GCC 4.8.4] on linux
Type "help", "copyright", "credits" or "license" for more information.
The text was updated successfully, but these errors were encountered: