Skip to content

Fix max SVs and protect against too many candidates

Thomas Boettcher requested to merge thboettc_fix_max_svs into master

This MR...

  • Removes a fudge factor of 10 in the maximum number of SVs in FilterTracks. This was a holdover from a previous method of handling SV combinatorics.
  • Adds a check to keep from overflowing the SV buffers in FilterTracks (@cagapopo's solution).

When we find more candidates than we can store, the current behavior is to set the number of candidates to zero. As opposed to naively truncating, this is a deterministic behavior. Something more elaborate can be developed later.

Edited by Rosen Matev

Merge request reports