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

VS-838. Remove the unneeded SCORE field from the filter_set_info_vqsr table #8278

Merged
merged 2 commits into from
Apr 5, 2023

Conversation

gbggrant
Copy link
Collaborator

@gbggrant gbggrant commented Apr 4, 2023

Remove the unneeded SCORE field from the filter_set_info_vqsr table.

The SCORE field was added to the filter_set_info_vqsr_lite table during debugging at Sam’s behest. It is the value that is used internally to determine the CALIBRATION_SENSITIVITY which is the primary ‘score’ of vqsr_lite. It can be removed now that we are done with debugging an issue we had earlier in development.

@codecov
Copy link

codecov bot commented Apr 4, 2023

Codecov Report

❗ No coverage uploaded for pull request base (ah_var_store@8217073). Click here to learn what that means.
The diff coverage is n/a.

Additional details and impacted files
@@               Coverage Diff                @@
##             ah_var_store     #8278   +/-   ##
================================================
  Coverage                ?   85.882%           
  Complexity              ?     35522           
================================================
  Files                   ?      2194           
  Lines                   ?    167029           
  Branches                ?     18005           
================================================
  Hits                    ?    143448           
  Misses                  ?     17200           
  Partials                ?      6381           

@gbggrant
Copy link
Collaborator Author

gbggrant commented Apr 5, 2023

Successful GvsCreateFilterSet run here and GvsExtract here

@gbggrant gbggrant marked this pull request as ready for review April 5, 2023 11:14
@@ -28,7 +28,6 @@ public final class GATKVCFConstants {
public static final String AS_CULPRIT_KEY = "AS_culprit";
public static final String AS_VQS_LOD_KEY = "AS_VQSLOD";
public static final String AS_VQS_SENS_KEY = "AS_VQS_SENS";
public static final String AS_VQS_SCORE_KEY = "AS_VQS_SCORE";
Copy link
Collaborator

Choose a reason for hiding this comment

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

oh cool this was the source of one of the compile errors when adding GVS files to Louis's candidate GVS-to-master branch 👋

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Happy to remove unnecessary, problematic code!

@gbggrant gbggrant merged commit 23a64a7 into ah_var_store Apr 5, 2023
@gbggrant gbggrant deleted the gg_VS-838_RemoveVQSRLiteScore branch April 5, 2023 17:01
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.

2 participants