-
Notifications
You must be signed in to change notification settings - Fork 8
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
Error in running given example #2
Comments
Hello, |
Merged
Hello, I still have the same problem after
|
I have the same problems |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Running the example as given with proxy files created with ProxyFinder, I got the following error:
Traceback (most recent call last):
File "./goshifter.py", line 187, in
args['--proxies'])
File "/specific/netapp5_2/elkon/avinoamshye/goshifter/functions.py", line 109, in enrichPermRandBoundryPeakShift_tabixLd
snpPeakInfo, ldsnpPeakInfo = permRandBoundryPeakShift(snpInfoChr,peaksTree,ldInfo,expand,minShift,maxShift,nPerm)
File "/specific/netapp5_2/elkon/avinoamshye/goshifter/functions.py", line 304, in permRandBoundryPeakShift
peaksToSnp = peaks2region(bounds,peaksTree,expand)
File "/specific/netapp5_2/elkon/avinoamshye/goshifter/functions.py", line 425, in peaks2region
maxbp = bounds[chrom][snp]['maxbp'] + expand
TypeError: unsupported operand type(s) for +: 'dict' and 'float'
It appears only some of the boundary values are still enclosed in dictionaries of the form {'bp': ... }
I could't find the reason for this so I added an if statement to extract the floats, assuming these values are the wanted ones.
This seem to work without any more errors, but I'm not sure the values inside the occasional dictionaries are correct. I'd love to hear your thoughts on this.
The text was updated successfully, but these errors were encountered: