mcveanlab / mccortex

De novo genome assembly and multisample variant calling
https://github.com/mcveanlab/mccortex/wiki
MIT License
113 stars 25 forks source link

missing tab means Bandage can't read mccortex GFA #26

Closed iqbal-lab closed 8 years ago

iqbal-lab commented 8 years ago

Not sure what the spec says, but Bandage expects a tab, not a space, after the initial H on line 1. Currently Mccortex outputs a space.

iqbal-lab commented 8 years ago

Actually Ryan Wick says the spec requires a tab

noporpoise commented 8 years ago

Just fixed in develop branch. Thanks Zam.

iqbal-lab commented 8 years ago

OK thanks! Should I used develop then in general?

noporpoise commented 8 years ago

Yes, probably best to use develop. Develop occasionally breaks but is more current. Master shouldn't break but is missing latest commits.