-
Notifications
You must be signed in to change notification settings - Fork 309
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Variants Visualization #542
Conversation
Test FAILed. Build result: ABORTEDGitHub pull request #542 of commit c11fec6 automatically merged.[EnvInject] - Loading node environment variables.Building remotely on amp-jenkins-worker-05 (centos) in workspace /home/jenkins/workspace/ADAM-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/adam.git # timeout=10Fetching upstream changes from https://github.com/bigdatagenomics/adam.git > git --version # timeout=10 > git fetch --tags --progress https://github.com/bigdatagenomics/adam.git +refs/pull/:refs/remotes/origin/pr/ > git rev-parse origin/pr/542/merge^{commit} # timeout=10Checking out Revision 2578aa3 (detached) > git config core.sparsecheckout # timeout=10 > git checkout -f 2578aa3 > git rev-list b409bd0004a00c2dac712b5c567963b3e013c8e3 # timeout=10Triggering ADAM-prb ? 2.2.0,centosTriggering ADAM-prb ? 1.0.4,centosTriggering ADAM-prb ? 2.3.0,centosADAM-prb ? 2.2.0,centos completed with result FAILUREADAM-prb ? 1.0.4,centos completed with result ABORTEDADAM-prb ? 2.3.0,centos completed with result FAILURETest FAILed. |
Jenkins, retest this please. |
@fnothaft We really need to fix that intermittent wigFix bug :-/ |
@laserson hah, yeah... I suppose we need to root-cause it first... |
Test FAILed. Build result: ABORTEDGitHub pull request #542 of commit c11fec6 automatically merged.[EnvInject] - Loading node environment variables.Building remotely on amp-jenkins-worker-05 (centos) in workspace /home/jenkins/workspace/ADAM-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/adam.git # timeout=10Fetching upstream changes from https://github.com/bigdatagenomics/adam.git > git --version # timeout=10 > git fetch --tags --progress https://github.com/bigdatagenomics/adam.git +refs/pull/:refs/remotes/origin/pr/ > git rev-parse origin/pr/542/merge^{commit} # timeout=10Checking out Revision 2578aa3 (detached) > git config core.sparsecheckout # timeout=10 > git checkout -f 2578aa3 > git rev-list b409bd0004a00c2dac712b5c567963b3e013c8e3 # timeout=10Triggering ADAM-prb ? 2.2.0,centosTriggering ADAM-prb ? 1.0.4,centosTriggering ADAM-prb ? 2.3.0,centosADAM-prb ? 2.2.0,centos completed with result FAILUREADAM-prb ? 1.0.4,centos completed with result SUCCESSADAM-prb ? 2.3.0,centos completed with result ABORTEDTest FAILed. |
@erictu Could you rebase this on master? |
} | ||
|
||
if (args.inputPath.endsWith(".vcf")) { | ||
VizReads.variants = sc.adamVCFLoad(args.inputPath).flatMap(_.genotypes) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Switch to loadGenotypes
?
A few small nits, but otherwise LGTM! |
Test PASSed. |
Test PASSed. |
Merged! Thanks @erictu! |
Working visualization of variants when given vcf input to ADAM viz
Also added loadGenotypes and maybeLoadVcf in ADAMContext