Commit 9e8c5e3a authored by André Anjos's avatar André Anjos 💬

Merge branch 'opt_test' into 'master'

Make use of test set on compute_perf optional; Update docs

This MR addresses two issues:

* Users which don't have a test set can now also use `bob_compute_perf`. In this case, only reduced number of statistics and plots are printed
* Fixes to the user guide, mainly concerning how to call the scripts since the simplifications with docopt

See merge request !22
parents ac41e56a e5324be0
Pipeline #5216 passed with stages
in 10 minutes and 24 seconds
This diff is collapsed.
......@@ -46,6 +46,22 @@ def test_compute_perf():
nose.tools.eq_(main(cmdline), 0)
def test_compute_perf_only_dev():
# sanity checks
assert os.path.exists(DEV_SCORES)
tmp_output = tempfile.NamedTemporaryFile(prefix=__name__, suffix='.pdf')
cmdline = [
DEV_SCORES,
'--output=' + tmp_output.name,
]
from .script.compute_perf import main
nose.tools.eq_(main(cmdline), 0)
def test_eval_threshold():
# sanity checks
......
This diff is collapsed.
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment