search

refresh-bio / Whisper

GNU General Public License v3.0
24 stars 4 forks source link

Option name doesn't match help text #12

Closed drtconway closed 2 years ago

drtconway commented 4 years ago

Line 73 of whisper.cpp prints the help text calling the option "-gzipped-SAM-level"

Line 171 matches against "-gzipped-SAM"

drtconway commented 4 years ago

Can I recommend using an options library, so you don't have duplicate definitions, and lots of duplicated code?

I've used boost program_options, which is ok.

Docopt is awesome as well - not quite as good in C++ as it is in Python, et al, but still an improvement.

agudys commented 4 years ago

Thanks for pointing this out! The number of options has grown so much that we definately need to handle command line parameters more systematically.

agudys commented 4 years ago

@drtconway The bug has been, hopefully fixed in the latest release.