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

Feat: move to python3 #242

Closed
vivekbhr opened this issue May 30, 2018 · 14 comments
Closed

Feat: move to python3 #242

vivekbhr opened this issue May 30, 2018 · 14 comments

Comments

@vivekbhr
Copy link

Hi @taoliu

What are the plans for moving macs2 code to python3?

@andersgs
Copy link

andersgs commented Jul 24, 2018

It is almost too easy to do it these days: http://python-future.org/automatic_conversion.html

@taoliu
Copy link
Contributor

taoliu commented Oct 24, 2018

Now there is a new macs2python3 branch I am working on to migrate MASC2 to Python3. It has passed all my tests under the 'test' folder. If you are interested in, you can install it and test it using your data. It might throw some errors in other situations such as those related to string vs. bytes types.

@nsheff
Copy link

nsheff commented Apr 25, 2019

any update on macs2 working with python3 ? when I try to install it in python 3 it says:

 Downloading https://files.pythonhosted.org/packages/5e/32/0108a85d9a91c18413e8492bea631725cdbc20b45970ef63e8d0020ef2da/MACS2-2.1.2.1.tar.gz (1.6MB)
    100% |████████████████████████████████| 1.6MB 7.2MB/s 
    Complete output from command python setup.py egg_info:
    CRITICAL: Python version must be 2.7!

@afrendeiro
Copy link

I just tested the macs2python3 branch with Python 3.7.3 and that works fine.
Any plans to merge the branch and submit to PyPI?

@whositwhatnow
Copy link

Where is the source for the python3 package? It's not on PyPI yet and I woudl liek to test it out

@taoliu
Copy link
Contributor

taoliu commented Aug 13, 2019

@afrendeiro I have to work on some testing scripts to test the branch macs2python3, and rebase this branch to the most recent v2.1.2 codes.

@taoliu taoliu self-assigned this Aug 13, 2019
@taoliu taoliu changed the title move to python3 Feat: move to python3 Aug 13, 2019
@taoliu taoliu added this to the Migration to Python3 milestone Aug 13, 2019
@taoliu
Copy link
Contributor

taoliu commented Aug 13, 2019

A new branch feat_py3 has been created by merging most recent master and macs2python3: 5afe82a

Work will be continued in the new branch...

@tillea
Copy link

tillea commented Sep 6, 2019

A Python3 port would be very interesting for the Debian packaged version. Debian is droping Python2 in the next release and we expect Python2 software to be removed in the not to distant future.
Kind regards, Andreas.

@taoliu taoliu pinned this issue Sep 19, 2019
@taoliu
Copy link
Contributor

taoliu commented Oct 1, 2019

MACS2 Python3 port is ready in the feat_py3 branch. Aim to release to PyPI by the end of this week.

PR is created #324

@tillea
Copy link

tillea commented Oct 1, 2019 via email

@taoliu
Copy link
Contributor

taoliu commented Oct 3, 2019

MACS v2.2.4 which is compatible with Python3 has been released.

Github

PyPI

During the testing with small datasets, v2.2.4 generates the same results as v2.1.4. If you find any inconsistency on your datasets or have any comments, feel free to open a new issue.

Thank everyone for your patience! I know that you are asking for a Python3 port of MACS for a long time.

@taoliu taoliu closed this as completed Oct 3, 2019
@taoliu taoliu unpinned this issue Oct 3, 2019
@tillea
Copy link

tillea commented Oct 3, 2019 via email

@taoliu
Copy link
Contributor

taoliu commented Oct 3, 2019

@tillea You are welcome! And as a long-time Debian user, thank you so much for helping MACS integrated into Debian! By the way, I think now cython can be safely removed from the dependencies of MACS.

@tillea
Copy link

tillea commented Oct 4, 2019 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants