bigdatagenomics / avocado

A Variant Caller, Distributed. Apache 2 licensed.
http://bdgenomics.org/projects/avocado/
Apache License 2.0
71 stars 42 forks source link

[AVOCADO-269] Add thresholds to DiscoverVariant CLI. #270

Closed fnothaft closed 6 years ago

fnothaft commented 6 years ago

Resolves #269. Adds phred and coverage thresholds to CLI, so that the DiscoverVariants CLI matches the BiallelicGenotyper CLI.

AmplabJenkins commented 6 years ago

Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/avocado-prb/216/

Build result: FAILURE

GitHub pull request #270 of commit c68efc94c15757fcc2ef122fd2551256aa7b6331 automatically merged.[EnvInject] - Loading node environment variables.Building remotely on amp-jenkins-staging-worker-02 (ubuntu staging-02 staging) in workspace /home/jenkins/workspace/avocado-prb > git rev-parse --is-inside-work-tree # timeout=10Fetching changes from the remote Git repository > git config remote.origin.url https://github.com/bigdatagenomics/avocado.git # timeout=10Fetching upstream changes from https://github.com/bigdatagenomics/avocado.git > git --version # timeout=10 > git fetch --tags --progress https://github.com/bigdatagenomics/avocado.git +refs/pull/:refs/remotes/origin/pr/ > git rev-parse origin/pr/270/merge^{commit} # timeout=10 > git branch -a -v --no-abbrev --contains 5f44ee72816097482ea3e934cbde45446d194e53 # timeout=10Checking out Revision 5f44ee72816097482ea3e934cbde45446d194e53 (origin/pr/270/merge) > git config core.sparsecheckout # timeout=10 > git checkout -f 5f44ee72816097482ea3e934cbde45446d194e53First time build. Skipping changelog.Triggering avocado-prb » 2.3.0,2.11,2.0.0,centosTriggering avocado-prb » 2.3.0,2.10,2.0.0,centosTriggering avocado-prb » 2.6.0,2.11,2.0.0,centosTriggering avocado-prb » 2.6.0,2.10,2.0.0,centosavocado-prb » 2.3.0,2.11,2.0.0,centos completed with result FAILUREavocado-prb » 2.3.0,2.10,2.0.0,centos completed with result FAILUREavocado-prb » 2.6.0,2.11,2.0.0,centos completed with result FAILUREavocado-prb » 2.6.0,2.10,2.0.0,centos completed with result FAILURE Test FAILed.

fnothaft commented 6 years ago

Jenkins, retest this please.

AmplabJenkins commented 6 years ago

Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/avocado-prb/217/ Test PASSed.