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

Exposed ability to blacklist intervals in CNV WDLs. #5027

Merged
merged 2 commits into from
Jul 25, 2018

Conversation

samuelklee
Copy link
Contributor

@samuelklee samuelklee commented Jul 18, 2018

This is a relatively simple change, but has not been tested. The reviewer should make sure I haven't screwed it up!

@samuelklee samuelklee requested a review from LeeTL1220 July 18, 2018 14:04
@codecov-io
Copy link

codecov-io commented Jul 18, 2018

Codecov Report

Merging #5027 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #5027   +/-   ##
=========================================
  Coverage     86.35%   86.35%           
  Complexity    28824    28824           
=========================================
  Files          1791     1791           
  Lines        133601   133601           
  Branches      14920    14920           
=========================================
  Hits         115364   115364           
  Misses        12834    12834           
  Partials       5403     5403

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.

@samuelklee Minor stuff...

@@ -1,5 +1,6 @@
task PreprocessIntervals {
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you add a doc comment?

@@ -28,6 +28,7 @@ workflow CNVSomaticPairWorkflow {
##################################
File common_sites
File intervals
File? blacklist_intervals
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you add a doc comment?

@@ -23,6 +23,7 @@ workflow CNVSomaticPanelWorkflow {
#### required basic arguments ####
##################################
File intervals
Copy link
Contributor

Choose a reason for hiding this comment

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

Ditto re: doc comment

@samuelklee samuelklee force-pushed the sl_add_XL_CNV_WDLs branch from 4c3524e to 57e51da Compare July 25, 2018 13:03
@samuelklee
Copy link
Contributor Author

Thanks, back to you @LeeTL1220!

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.

@samuelklee Looks good to me.

@samuelklee samuelklee merged commit dca22bc into master Jul 25, 2018
@samuelklee samuelklee deleted the sl_add_XL_CNV_WDLs branch July 25, 2018 14:23
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