-
Notifications
You must be signed in to change notification settings - Fork 597
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
Adding oncotator filtering enabled in M2 WDL. #4423
Conversation
Codecov Report
@@ Coverage Diff @@
## master #4423 +/- ##
==============================================
+ Coverage 79.04% 79.402% +0.362%
- Complexity 16447 17620 +1173
==============================================
Files 1047 1055 +8
Lines 59189 63340 +4151
Branches 9672 10754 +1082
==============================================
+ Hits 46783 50293 +3510
- Misses 8645 9125 +480
- Partials 3761 3922 +161
|
@davidbenjamin I also ran an additional test outside of our repo to make sure that this was working properly. |
@brigranger @jnktsj Note when this gets into master and gets released. To keep the previous behavior, you will want to set (The default is going to become |
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.
Looks good!
Fixes #4422