csf-ngs / surecellumi

extraction of Illumina SureCell UMI protocol. Compatible with umi-tools
1 stars 0 forks source link

error for running mergeUmi #1

Open lixin4306ren opened 7 years ago

lixin4306ren commented 7 years ago

Thank you for providing this extractor. My python version is 3.5.2. I instlled the package and then tried to run merge.sh and got following error:

2017-08-31 03:40:48,358 - root - INFO - starting extracting umis
2017-08-31 03:40:48,359 - root - INFO - extracting umis with: TAGCCATCGCATTGC TACCTCTGAGCTGAA TAGCCATCGCATTGCNNNNNNTACCTCTGAGCTGAA
Traceback (most recent call last):
  File "/home/others/xli/bin/mergeUmi", line 6, in <module>
    exec(compile(open(__file__).read(), __file__, 'exec'))
  File "/home/others/xli/soft/src/surecellumi/bin/mergeUmi", line 28, in <module>
    mergeUmiFiles(args.r1, args.r2, args.outfq, args.outextract, args.outbc, args.outprime5, ll1, ll2, llinker)
  File "/home/others/xli/soft/src/surecellumi/surecellumi/mergefq.py", line 26, in mergeUmiFiles
    ignore = f1.next()
AttributeError: '_io.TextIOWrapper' object has no attribute 'next'
idot commented 7 years ago

Thank you for trying. I have disabled python 3 for this package. The python 2 version compiles and passes all tests.