md5sam / Falcon2Fastg

Falcon2Fastg is a tool for converting a FALCON assembly to FASTG format to visualize with Bandage
MIT License
13 stars 1 forks source link

Remove double edges #7

Closed rchikhi closed 9 years ago

rchikhi commented 9 years ago

I had a look at the test/ data. It feels too weird to have a graph with double edges there :) Don't you think? We need to fix the double edges issue before release

md5sam commented 9 years ago

Agreed, I need to think more about how to resolve these double edges. If Falcon itself is reporting a double edge (by connecting B to B and E to E for every pair), then I need to think of maybe an alternative way of representing this (possibly redundant) information.

Let's move the release to early next week ?

rchikhi commented 9 years ago

Yep lets see on monday.

rchikhi commented 9 years ago

Have a look at the small example, sg_edges_list:

000000001:B 000000000:B 000000000 4000 0 1000 100.0 G 000000002:B 000000001:B 000000001 3000 0 1000 100.0 G 000000000:E 000000001:E 000000001 1000 4000 1000 100.0 G 000000001:E 000000002:E 000000002 1000 3000 1000 100.0 G

rchikhi commented 9 years ago

Fixed as of today's patch