-
Notifications
You must be signed in to change notification settings - Fork 29
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
Add variants_filter_lcr
, variants_filter_segdup
and variants_snv_only
options to annotate_sex
to filter variants prior to variant only ploidy imputation
#479
Conversation
…ref or var specific ploidy estimates
…nt_filter_options' of https://github.com/broadinstitute/gnomad_methods into jg/annotate_sex_variant_filter_options
…nt_filter_options' of https://github.com/broadinstitute/gnomad_methods into jg/annotate_sex_variant_filter_options
…nt_filter_options' of https://github.com/broadinstitute/gnomad_methods into jg/annotate_sex_variant_filter_options � Conflicts: � gnomad/sample_qc/pipeline.py
…nt_filter_options' of https://github.com/broadinstitute/gnomad_methods into jg/annotate_sex_variant_filter_options
…ref or var specific ploidy estimates
…ub.com/broadinstitute/gnomad_methods into jg/annotate_sex_variant_filter_options
"x_ploidy": "chrX_ploidy", | ||
"y_ploidy": "chrY_ploidy", | ||
"x_mean_dp": "chrX_mean_dp", | ||
"y_mean_dp": "chrY_mean_dp", |
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.
is this in general the names we want to use or it is based on the reference used?
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.
This is what we used for UKBB and v3, so I was keeping the output consistent since this function was used in both of those repos
Co-authored-by: klaricch <kristen@broadinstitute.org>
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.
lgtm
NOTE: This PR is stacked on the add_gaussian_karyotype PR #478
If helpful for testing: