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

AssertionError #18

Open
mzwaig opened this issue Mar 12, 2020 · 2 comments · May be fixed by #23
Open

AssertionError #18

mzwaig opened this issue Mar 12, 2020 · 2 comments · May be fixed by #23

Comments

@mzwaig
Copy link

mzwaig commented Mar 12, 2020

Hello,

I had the same issue as in #17 but I managed to run NAIBR on most of my samples by giving it lost of processors (16 or more) but only 1 threads.

For one sample, I'm getting an additional error (assert len(lengths)>100) and I'm not sure why.

Here's the log file

Traceback (most recent call last):
File "/lb/project/Melissa-abacus/tools/NAIBR/NAIBR.py", line 90, in
main()
File "/lb/project/Melissa-abacus/tools/NAIBR/NAIBR.py", line 61, in main
data = parallel_execute(run_NAIBR,chroms)
File "/lb/project/Melissa-abacus/tools/NAIBR/src/utils.py", line 395, in parallel_execute
data = map(function,input_list)
File "/lb/project/Melissa-abacus/tools/NAIBR/NAIBR.py", line 35, in run_NAIBR
cands,p_len,p_rate = get_candidates(discs,reads_by_LR)
File "/lb/project//Melissa-abacus/tools/NAIBR/src/get_reads.py", line 154, in get_candidates
p_len,p_rate,barcode_overlap = get_distributions(reads_by_LR)
File "/lb/project/Melissa-abacus/tools/NAIBR/src/distributions.py", line 120, in get_distributions
p_len = get_length_distr(LRs)
File "/lb/project/Melissa-abacus/tools/NAIBR/src/distributions.py", line 126, in get_length_distr
assert len(lengths)>100
AssertionError

Thanks for you help,
Melissa

@zuzanahalenkova
Copy link

Hello, I have encountered the same error, did you manage to overcome it somehow?
Thank you for your help,
Zuzana

pontushojer added a commit to pontushojer/NAIBR that referenced this issue Apr 19, 2022
@pontushojer pontushojer linked a pull request Apr 19, 2022 that will close this issue
@pontushojer
Copy link

@zuzanahalenkova I implemented a fix to what possibly could be the issue here in the PR #23. Try it out and see if that works.

pontushojer added a commit to pontushojer/NAIBR that referenced this issue Aug 26, 2022
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

Successfully merging a pull request may close this issue.

3 participants