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

Updated the experimental funcotator.wdl to work properly in cromwell. #4870

Merged
merged 2 commits into from
Jun 8, 2018

Conversation

jonn-smith
Copy link
Collaborator

Tested with Cromwell 30.2

Copy link
Contributor

@LeeTL1220 LeeTL1220 left a comment

Choose a reason for hiding this comment

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

Very minor. If you address, feel free to merge.


String? data_sources_tar_gz
File? interval_list
Copy link
Contributor

Choose a reason for hiding this comment

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

Need to add a comment up top for interval_list parameter. Make sure to mention that it is optional.

String? transcript_selection_mode
Array[String]? transcript_selection_list
Array[String]? annotation_defaults
Array[String]? annotation_overrides
File? gatk4_jar_override

call MakeItFunky {
String? funcotator_extra_args
Copy link
Contributor

Choose a reason for hiding this comment

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

Need to add a comment up top for funcotator_extra_args

@codecov-io
Copy link

codecov-io commented Jun 8, 2018

Codecov Report

Merging #4870 into master will increase coverage by 0.108%.
The diff coverage is n/a.

@@              Coverage Diff               @@
##             master     #4870       +/-   ##
==============================================
+ Coverage     80.42%   80.528%   +0.108%     
- Complexity    17813     18352      +539     
==============================================
  Files          1089      1089               
  Lines         64133     65588     +1455     
  Branches      10340     10743      +403     
==============================================
+ Hits          51576     52817     +1241     
- Misses         8499      8649      +150     
- Partials       4058      4122       +64
Impacted Files Coverage Δ Complexity Δ
...utils/smithwaterman/SmithWatermanIntelAligner.java 50% <0%> (-30%) 1% <0%> (-2%)
...nder/tools/funcotator/TranscriptSelectionMode.java 89.326% <0%> (-0.394%) 1% <0%> (ø)
.../tools/funcotator/mafOutput/MafOutputRenderer.java 96.289% <0%> (+0.648%) 87% <0%> (+36%) ⬆️
...e/hellbender/tools/funcotator/FuncotatorUtils.java 81.068% <0%> (+0.759%) 320% <0%> (+157%) ⬆️
...org/broadinstitute/hellbender/engine/GATKTool.java 91.463% <0%> (+0.797%) 108% <0%> (+9%) ⬆️
.../tools/funcotator/vcfOutput/VcfOutputRenderer.java 86.17% <0%> (+0.924%) 18% <0%> (+6%) ⬆️
...titute/hellbender/tools/funcotator/Funcotator.java 86.638% <0%> (+1.604%) 65% <0%> (+24%) ⬆️
...dataSources/gencode/GencodeFuncotationFactory.java 85.104% <0%> (+1.745%) 268% <0%> (+107%) ⬆️
...otator/dataSources/gencode/GencodeFuncotation.java 65.353% <0%> (+2.656%) 368% <0%> (+147%) ⬆️
...cotator/dataSources/vcf/VcfFuncotationFactory.java 90% <0%> (+2.987%) 25% <0%> (ø) ⬇️
... and 5 more

@jonn-smith jonn-smith merged commit 1a7b0b7 into master Jun 8, 2018
@jonn-smith jonn-smith deleted the jts_funcotator_wdl_fix branch June 8, 2018 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants