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

INSTALL_b38.md :: ClinSV requires python2 AND python3 #40

Open
EricDeveaud opened this issue Jan 19, 2023 · 1 comment
Open

INSTALL_b38.md :: ClinSV requires python2 AND python3 #40

EricDeveaud opened this issue Jan 19, 2023 · 1 comment

Comments

@EricDeveaud
Copy link

Hello,

I noticed in the install from sources documentation, more specifically in install python section.
https://github.com/KCCG/ClinSV/blob/master/INSTALL_b38.md#install-python

python2 is required
clinsv uses python2 for lumpy calls
lumpy requires numpy pysam

python3 is required to
clinsv uses python3 for pairend_distro-a1.py see: line 460. of clinsv requires numpy
NB I did not check if pairend_distro-a1.py use some python3 specific syntax, except for print
maybee it means that clinsv can rely only in python2 using from future import print

regards

Eric

@drmjc
Copy link
Member

drmjc commented Feb 12, 2023

we are keen to drop all the py2 code in favour of py3... seamlessly supporting x38 was higher on the list, but we'll get to this one day

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

2 participants