search

SciLifeLab / facs

Fast and Accurate Classification of Sequences using Bloom filters
http://facs.scilifelab.se/
Other
16 stars 9 forks source link

facs classify -h #19

Closed henrikstranneheim closed 11 years ago

henrikstranneheim commented 11 years ago
  1. wrong header says "./facs remove" should say "./facs classify"
  2. -t tolerance rate, default is 0.0005. See issue #15.
  3. There is a bad address message when running facs classify -h.
tzcoolman commented 11 years ago

@henrikstranneheim Why?? This script is for sequence decontamination I think. The real ./facs classify is under testing

henrikstranneheim commented 11 years ago

Please specify what you are saying why to. Even if facs classify is under functional testing the help text needs correction at some point. That why this issue was reported.

tzcoolman commented 11 years ago

okay... i ll write a help doc 1st @henrikstranneheim

tzcoolman commented 11 years ago

@henrikstranneheim I have fixed

  1. threshold value instead of tolerant rate
  2. Capital B for Bloom filter
  3. ./facs classify -h
  4. Other bad flags . Need to w8 for Roman to double check so that it can finally be merged to scilifelab/facs