Skip to content
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

Changing variable name in MT WDL to match gatk-workflows #5694

Merged
merged 1 commit into from
Feb 21, 2019

Conversation

meganshand
Copy link
Contributor

Small change to match the WDL that's going into gatk-workflows.

@meganshand meganshand changed the title Changing variable name to match gatk-workflows Changing variable name in MT WDL to match gatk-workflows Feb 20, 2019
@codecov-io
Copy link

codecov-io commented Feb 20, 2019

Codecov Report

Merging #5694 into master will decrease coverage by 69.096%.
The diff coverage is n/a.

@@               Coverage Diff                @@
##              master     #5694        +/-   ##
================================================
- Coverage     87.068%   17.971%   -69.096%     
+ Complexity     31855      8570     -23285     
================================================
  Files           1940      1940                
  Lines         146679    146679                
  Branches       16218     16218                
================================================
- Hits          127710     26360    -101350     
- Misses         13057    117642    +104585     
+ Partials        5912      2677      -3235
Impacted Files Coverage Δ Complexity Δ
...ls/variant/writers/GVCFBlockCombiningIterator.java 0% <0%> (-100%) 0% <0%> (-1%)
...nder/tools/copynumber/utils/TagGermlineEvents.java 0% <0%> (-100%) 0% <0%> (-3%)
...r/tools/spark/pathseq/PSBwaArgumentCollection.java 0% <0%> (-100%) 0% <0%> (-1%)
...titute/hellbender/utils/mcmc/DecileCollection.java 0% <0%> (-100%) 0% <0%> (-5%)
...ender/tools/readersplitters/ReadGroupSplitter.java 0% <0%> (-100%) 0% <0%> (-3%)
...ynumber/models/AlleleFractionGlobalParameters.java 0% <0%> (-100%) 0% <0%> (-10%)
...ls/walkers/genotyper/HeterogeneousPloidyModel.java 0% <0%> (-100%) 0% <0%> (-14%)
...rs/vqsr/VariantRecalibratorArgumentCollection.java 0% <0%> (-100%) 0% <0%> (-1%)
...adinstitute/hellbender/engine/ReferenceWalker.java 0% <0%> (-100%) 0% <0%> (-9%)
...ools/funcotator/filtrationRules/ClinVarFilter.java 0% <0%> (-100%) 0% <0%> (-7%)
... and 1639 more

Copy link

@bshifaw bshifaw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@meganshand meganshand merged commit 2500cd3 into master Feb 21, 2019
@meganshand meganshand deleted the ms_var_name branch February 21, 2019 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants