-
Notifications
You must be signed in to change notification settings - Fork 596
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
Conversation
Codecov Report
@@ 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 |
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.
@samuelklee Minor stuff...
@@ -1,5 +1,6 @@ | |||
task PreprocessIntervals { |
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.
Can you add a doc comment?
@@ -28,6 +28,7 @@ workflow CNVSomaticPairWorkflow { | |||
################################## | |||
File common_sites | |||
File intervals | |||
File? blacklist_intervals |
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.
Can you add a doc comment?
@@ -23,6 +23,7 @@ workflow CNVSomaticPanelWorkflow { | |||
#### required basic arguments #### | |||
################################## | |||
File intervals |
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.
Ditto re: doc comment
4c3524e
to
57e51da
Compare
Thanks, back to you @LeeTL1220! |
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.
@samuelklee Looks good to me.
This is a relatively simple change, but has not been tested. The reviewer should make sure I haven't screwed it up!