-
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
Made number of samples to put variant site in M2 pon adjustable #4566
Conversation
Codecov Report
@@ Coverage Diff @@
## master #4566 +/- ##
==============================================
+ Coverage 79.845% 79.85% +0.005%
Complexity 16964 16964
==============================================
Files 1062 1062
Lines 61676 61677 +1
Branches 9983 9983
==============================================
+ Hits 49245 49249 +4
+ Misses 8539 8536 -3
Partials 3892 3892
|
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.
One comment. No need for a second review
@@ -114,6 +118,14 @@ | |||
doc="VCFs for samples to include. May be specified either one at a time, or as one or more .args file containing multiple VCFs, one per line.", optional = false) | |||
private Set<File> vcfs = new LinkedHashSet<>(0); | |||
|
|||
/** | |||
* The VCFs can be input as either one or more .args file(s) containing one VCF per line, or VCFs can be |
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.
I think this comment block got copied and pasted from above
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.
Whoops!
@takutosato Can you review this?
@sooheelee This closes #4552.