search

KCCG / ClinSV

Robust detection of clinically relevant structural and copy number variation from whole genome sequencing data
Other
61 stars 8 forks source link

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

Open EricDeveaud opened 1 year ago

EricDeveaud commented 1 year ago

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 commented 1 year ago

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