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

Change get_downsampling_freq_indices and downsampling_counts_expr to support both 'pop' and 'gen_anc' keys in metadata #633

Merged
merged 17 commits into from
Apr 8, 2024

Conversation

jkgoodrich
Copy link
Contributor

Needed for some plots I am creating that use these two functions on the v4 data

… to support both 'pop' and 'gen_anc' keys inf metadata
@jkgoodrich
Copy link
Contributor Author

This is back to me, but is lower in priority than others.

@jkgoodrich jkgoodrich marked this pull request as draft January 4, 2024 15:55
@jkgoodrich jkgoodrich marked this pull request as ready for review April 3, 2024 16:47
gnomad/utils/constraint.py Outdated Show resolved Hide resolved
gnomad/utils/constraint.py Outdated Show resolved Hide resolved
gnomad/utils/constraint.py Outdated Show resolved Hide resolved
gnomad/utils/constraint.py Outdated Show resolved Hide resolved
gnomad/utils/constraint.py Outdated Show resolved Hide resolved
gnomad/utils/constraint.py Outdated Show resolved Hide resolved
gnomad/utils/constraint.py Outdated Show resolved Hide resolved
klaricch and others added 6 commits April 4, 2024 12:41
Co-authored-by: jkgoodrich <33063077+jkgoodrich@users.noreply.github.com>
Co-authored-by: jkgoodrich <33063077+jkgoodrich@users.noreply.github.com>
Co-authored-by: jkgoodrich <33063077+jkgoodrich@users.noreply.github.com>
Co-authored-by: jkgoodrich <33063077+jkgoodrich@users.noreply.github.com>
Copy link
Contributor Author

@jkgoodrich jkgoodrich left a comment

Choose a reason for hiding this comment

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

LGTM

@klaricch klaricch self-requested a review April 8, 2024 13:59
@klaricch klaricch merged commit fde54e4 into main Apr 8, 2024
3 checks passed
@klaricch klaricch deleted the jg/modify_constraint_pop_for_gen_anc branch April 8, 2024 14:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants