andersen-lab / ivar

iVar is a computational package that contains functions broadly useful for viral amplicon-based sequencing.
https://andersen-lab.github.io/ivar/html/
GNU General Public License v3.0
115 stars 39 forks source link

Clarification of new ivar consensus -c (minimum insertion frequency) parameter #149

Open wm75 opened 1 year ago

wm75 commented 1 year ago

The command line help for this param currently reads: -c Minimum insertion frequency threshold(0 - 1) to call consensus. and miminum_insertion_frequency is also the internal name used for the supplied value, but looking at the code it seems that this is actually treated like a minimum_indel_frequency, which also seems like the expected behavior for such an option? So should the help message get adjusted and the internal variable be renamed?