Skip to content

GnomadMT plugin ENSVAR-6912#820

Merged
ainefairbrother merged 21 commits intoEnsembl:mainfrom
mluypaert:gnomad-mt_ENSVAR-6912
Feb 16, 2026
Merged

GnomadMT plugin ENSVAR-6912#820
ainefairbrother merged 21 commits intoEnsembl:mainfrom
mluypaert:gnomad-mt_ENSVAR-6912

Conversation

@mluypaert
Copy link
Member

No description provided.

@mluypaert mluypaert marked this pull request as ready for review January 9, 2026 14:26
@mluypaert mluypaert marked this pull request as draft January 14, 2026 13:06
@mluypaert
Copy link
Member Author

Converted back to draft as needing rework to take gnomAD data from the complete vcf (incl haplotype info) instead of the reduced tsv file that is used atm.

@mluypaert mluypaert force-pushed the gnomad-mt_ENSVAR-6912 branch from f65cc01 to 3079a5a Compare January 29, 2026 12:28
@mluypaert mluypaert force-pushed the gnomad-mt_ENSVAR-6912 branch from 42d8bef to edb4ae7 Compare February 10, 2026 13:48
@mluypaert mluypaert marked this pull request as ready for review February 10, 2026 13:49
@ainefairbrother ainefairbrother self-requested a review February 11, 2026 10:05
Copy link
Contributor

@ainefairbrother ainefairbrother left a comment

Choose a reason for hiding this comment

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

Thanks for this plugin @mluypaert - some potential bugs highlighted as well as a couple of suggestions.

@ainefairbrother ainefairbrother self-assigned this Feb 12, 2026
@mluypaert
Copy link
Member Author

@ainefairbrother Thanks for the feedback and suggestions. I've implemented all of them, so this should now be ready for approval.

Copy link
Contributor

@ainefairbrother ainefairbrother left a comment

Choose a reason for hiding this comment

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

Thanks for those changes @mluypaert, looks good to me!

Couple of tiny text things, then good to go.

Copy link
Contributor

@ainefairbrother ainefairbrother left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks @mluypaert

@ainefairbrother ainefairbrother merged commit d85f1ac into Ensembl:main Feb 16, 2026
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