Skip to content
This repository was archived by the owner on Mar 16, 2022. It is now read-only.

Fix fetch_reads#493

Open
pb-cdunn wants to merge 2 commits intomasterfrom
fetch_reads
Open

Fix fetch_reads#493
pb-cdunn wants to merge 2 commits intomasterfrom
fetch_reads

Conversation

@pb-cdunn
Copy link

From DNANexus, privately.

Jason, this was the most recent code exchanged. It has not caused me any problems, but I have not looked into this matter very deeply either.

From DNANexus, privately.
This is for a different problem, discussed between Jason and Dario. I am not certain of the problem yet. From Jason's email:

For FALCON-unzip, you can comment out the following lines in graph_to_contig.py to avoid the seg fault. The FALCON-Unzip will run. Your a_ctg.fa will have some duplicates but it will not affects downstream in general. The detail of a_tig is not used in Unzip process as it replaces it with phased haplotigs.

Ok, the root cause is that there is a very long bubble.  The alternative path is > 500kb.  The internal aligner used is not designed for handling such long sequences. In a halpoid genome, this is not supposed to happened. However, in Cab-Suv genome, we might generate big bubble in the graph level.  One proposed fix is to cap the size of sequence sent to the internal aligner now.
@pb-cdunn pb-cdunn force-pushed the master branch 2 times, most recently from baf9db6 to 58083fe Compare February 11, 2017 20:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant