nmdp-bioinformatics / pipeline

Consensus assembly and allele interpretation pipeline.
GNU Lesser General Public License v3.0
7 stars 7 forks source link

Add option to remove alignment gaps '~' in the filtered consensus sequen... #68

Closed ckennedy-nmdp closed 9 years ago

ckennedy-nmdp commented 9 years ago

...ce; fixes bug #67 where deletions are replaced with 'N' but not removed so truncated alleles are not interpreted properly

ghost commented 9 years ago

Since there are no unit or functional tests for this code, might you provide an example as a comment to this pull request that was broken before and works now? I'm not sure where the ~s are coming from, that's not a typical biojava gap symbol.

ckennedy-nmdp commented 9 years ago

Sure, I'll detail in bug #67.

ghost commented 9 years ago

Thanks @ckennedy-nmdp

ckennedy-nmdp commented 9 years ago

Thanks!