-
Notifications
You must be signed in to change notification settings - Fork 31
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 [result_end_clip >= 0] in resolve_triplets_kmerify.py #99
Comments
This seems like a bug in verkko and without the data, it'd be hard to diagnose exactly what is happening. Are you able to share at least the 4-processONT folder, there are instructions on how to send us data: https://canu.readthedocs.io/en/latest/faq.html#how-can-i-send-data-to-you |
Thanks for the answer. /usr/local/lib/verkko/scripts/resolve_triplets_kmerify.py \
normal-connected.gfa \ #input_gfa = sys.argv[1]
fake-ont-paths.txt \ #out_path_file = sys.argv[2]
fake-ont-nodecovs.csv \ #node_coverage_file = sys.argv[3]
resolve-mapping.txt \ #resolve_namemapping_file = sys.argv[4]
100000 \ #max_resolve_length = int(sys.argv[5])
5 \ #min_allowed_coverage = float(sys.argv[6])
20 10 5 \ #resolve_steps = [int(n) for n in sys.argv[7:]]
< fake-ont-alns.gaf \
> ont-resolved-graph.gfa \
2> ont_resolved_graph.gfa.err Due to the overall size for the moment I shared via ftp the files:
( Hope this helps and please let me know if you need any additional file or piece of information. Luca |
I just committed a potential fix for this. You could be able to replace the relevant python script with the new version and see if it continues without error. |
Ok now it worked. Thanks! |
Hi,
I renew all the files in scripts as you mentioned above, but got the same error. The current version of verkko is 1.2. Could you please help check this? I will attach the 4-processONT below. |
I can't upload the 4-processONT folder via ftp due to connection error. You may find the files in https://drive.google.com/file/d/198bPbNkcsEGOuca5lgTfZNQaERRj8RNk/view?usp=sharing I will also re-run the verkko v1.3.1 and update results here. |
update here, I got the same error at step16 of 4-processONT using v1.3.1. |
@skoren could you please help check this? Thank you! |
I can reproduce the error locally but not sure what the cause is, it looks like the script may have an off by one error. @maickrau could you take a look. The data is in globus under issue99. The node utig1-19176 seems to end up with a 0bp extension. |
This was caused by the resolution script not handling hairpin repeats (perfect palindromes) correctly, it should be fixed in fe83d29 |
Issue99_update.zip |
@maickrau could you please check this new result again? can you successfully output ont-resolved-graph.gfa? Best, |
I was able to run your dataset to completion with the updated code and originally shared files, yes. That error sounds like something is incorrect in the script or the replaced file didn't get installed in the verkko library folder. The step 16 code was:
The final gfa is about 1.1 GB in size. Try running the above (with the proper path to your resolve_triplets_kmerify.py script by hand and see what it reports. |
Hi, thanks very much for help! Previously there was a window-format issue, it works well now. Best, |
I ran into this error in the processONT stage:
Could you please provide any hint regarding how to circumvent this? the input files are too large to share, but please let me know should I inspect the logs further and/or provide other intermediate files.
Thanks!
The text was updated successfully, but these errors were encountered: