BUGFIX: This resolves a corner case.

NCBI-Hackathons / NovoGraph

NovoGraph: building whole genome graphs from long-read-based de novo assemblies

MIT License

44 stars 8 forks source link

BUGFIX: This resolves a corner case. #35

Closed TorHou closed 4 years ago

TorHou commented 4 years ago

The problem occurred when minWindowPos was within chromosome limits, but the coverage was 0. That lead to middleWindowPos being added to the window_positions array which could be bigger than the chromosomes length.

evanbiederstedt commented 4 years ago

Ah, it was the wrong variable!

Good catch!

Did it work? I'll merge, and if there issues, we can revisit.