vgteam / vg

tools for working with genome variation graphs
https://biostars.org/tag/vg/
Other
1.07k stars 191 forks source link

Remove limit on Dozeu problem cell count from old Giraffe codepath #4308

Open adamnovak opened 2 weeks ago

adamnovak commented 2 weeks ago

Even though we now use a Dozeu with a working allocator in vg's master branch, we also still limit the total number of Dozeu DP calls, ostensibly to work around the old, broken allocator: https://github.com/vgteam/vg/blob/4b1c72ba190093799895d786015eef74d7485550/src/minimizer_mapper.hpp#L338-L344

We should remove the limit and make sure that that doesn't cause speed or accuracy regressions, or at least bring it out to the command line.