-
Notifications
You must be signed in to change notification settings - Fork 589
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
VS-1009 Add SCORE to VQSR Lite VCF #8423
Conversation
- Pull it from the VCFs exported from VQSR Lite and store it in the model
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## ah_var_store #8423 +/- ##
================================================
Coverage ? 16.791%
Complexity ? 4708
================================================
Files ? 1392
Lines ? 83169
Branches ? 13248
================================================
Hits ? 13965
Misses ? 67144
Partials ? 2060 |
Github actions tests reported job failures from actions build 5582472783
|
Github actions tests reported job failures from actions build 5590654477
|
Github actions tests reported job failures from actions build 5602967249
|
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 small question, otherwise LGTM 👍🏻
src/main/java/org/broadinstitute/hellbender/tools/gvs/extract/ExtractCohortEngine.java
Outdated
Show resolved
Hide resolved
* Add the VQSR Lite field 'SCORE' to the data model - Pull it from the VCFs exported from VQSR Lite and store it in the model * Add code to put SCORE in the extracted VCF * Pass the gatk_override to VQSRClassic wdl (oops) * Update truth for integration test to include VQSR Lite VCFs with 'SCORE' field in them * Update VQSR Lite avro file and test data to include SCORE field. * Updated dockers and override jar
This PR adds the 'SCORE' field as an output in the VQSR-Lite derived VCFs
Score is the value from which the
CALIBRATION_SENSITIVITY
is derived. The latter is what we use for filtering based on sensitivity, but Sam and Laura also want the SCORE stored in the VCF.Passing Integration test here