search

amplab / snap

Scalable Nucleotide Alignment Program -- a fast and accurate read aligner for high-throughput sequencing data
https://www.microsoft.com/en-us/research/project/snap/
Apache License 2.0
288 stars 66 forks source link

usage not completely printing for indexing option #30

Closed joecop closed 10 years ago

joecop commented 10 years ago

Hi, I have noticed a sneaky comma at the end of the -bSpace description in the usage for index option. See the file snap/SNAPLib/GenomeIndex.cpp line 64 This is not a big issue and i am not sure if the comma is there intentionally but it does cause the last 4 options (-pPadding, -HHistogramFile, -exact, -keysize) from not printing when the usage is printed.

bolosky commented 10 years ago

Good catch, it is incorrect.

We’ve already fixed it in the dev branch, so it’ll be fixed the next time we release a beta.

--Bill

From: joecop [mailto:notifications@github.com] Sent: Wednesday, August 27, 2014 11:55 PM To: amplab/snap Subject: [snap] usage not completely printing for indexing option (#30)

Hi, I have noticed a sneaky comma at the end of the -bSpace description in the usage for index option. See the file snap/SNAPLib/GenomeIndex.cpp line 64 This is not a big issue and i am not sure if the comma is there intentionally but it does cause the last 4 options (-pPadding, -HHistogramFile, -exact, -keysize) from not printing when the usage is printed.

— Reply to this email directly or view it on GitHubhttps://github.com/amplab/snap/issues/30.

joecop commented 10 years ago

OK great. Thanks.