search

bacpop / PopPUNK

PopPUNK 👨‍🎤 (POPulation Partitioning Using Nucleotide Kmers)
https://www.bacpop.org/poppunk
Apache License 2.0
91 stars 18 forks source link

Fix option names for k-mer range min/max and k-mer step. #239

Closed tmaklin closed 1 year ago

tmaklin commented 1 year ago

Hi, I noticed a few typos in the documentation for --create-db when trying to run PopPUNK with variable k-mer lengths. The current documentation mentions using the --k-min, --k-max, and --kmer-step options but I think these should be --min-k, --max-k, and --k-step which are what poppunk --create-db actually accepts. Using the old options results in an unrecognized argument error.

johnlees commented 1 year ago

Thanks Tommi, I will add these fixes into #240 (a bit easier with CI)