bcgsc / biobloom

Create Bloom filters for a given reference and then use it to categorize sequences
http://www.bcgsc.ca/platform/bioinfo/software/biobloomtools
GNU General Public License v3.0
75 stars 15 forks source link

core dumps for long parameter names #13

Closed zyxue closed 8 years ago

zyxue commented 8 years ago
$ biobloomcategorizer \
    --prefix experiment/case1/tasrkleat-results/biobloomcategorizer/cba \
    --paired_mode \
    --score 0.6 \
    --with_score \
    --inclusive \
    --filter_files 'experiment/utrtargets/bf/combined.bf' \
    --threads 4 \
    --fq experiment/case1/tasrkleat-results/extract_tarball/cba_1.fastq experiment/case1/tasrkleat-results/extract_tarball/cba_2.fastq
Segmentation fault (core dumped)

Changing --prefix to -p causes a different error

$ biobloomcategorizer \
  -p experiment/case1/tasrkleat-results/biobloomcategorizer/cba \
  --paired_mode \
  --score 0.6 \
  --with_score \
  --inclusive \
  --filter_files 'experiment/utrtargets/bf/combined.bf' \
  --threads 4 \
  --fq experiment/case1/tasrkleat-results/extract_tarball/cba_1.fastq experiment/case1/tasrkleat-results/extract_tarball/cba_2.fastq
terminate called after throwing an instance of 'std::logic_error'
what():  basic_string::_S_construct null not valid
Aborted (core dumped)

Changing --score to -s makes the error gone

$ biobloomcategorizer \
  -p experiment/case1/tasrkleat-results/biobloomcategorizer/cba \
  --paired_mode \
  -s 0.6 \
  --with_score \
  --inclusive \
  --filter_files 'experiment/utrtargets/bf/combined.bf' \
  --threads 4 \
  --fq experiment/case1/tasrkleat-results/extract_tarball/cba_1.fastq experiment/case1/tasrkleat-results/extract_tarball/cba_2.fastq
Starting to Load Filters.
Loaded Filter: combined
Filter Loading Complete.
...
zyxue commented 8 years ago
$ biobloomcategorizer --version
biobloomcategorizer (BIOBLOOMTOOLS) 2.0.13b
Written by Justin Chu.

Copyright 2013 Canada's Michael Smith Genome Science Centre
JustinChu commented 8 years ago

Thanks for the heads up, I just pushed a fix I hope fixes the problem.

b439035d3c6ddb5ab7464a4fab38ded4b5355f72