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

feat: updated .vcf.gz output in simgenotype #150

Merged
merged 8 commits into from
Dec 22, 2022
Merged

feat: updated .vcf.gz output in simgenotype #150

merged 8 commits into from
Dec 22, 2022

Conversation

mlamkin7
Copy link
Collaborator

@mlamkin7 mlamkin7 commented Dec 21, 2022

resolves #148

Updated VCF output to output as a .gz file

@mlamkin7 mlamkin7 requested a review from aryarm December 21, 2022 19:55
@aryarm aryarm changed the title feat Updated VCF output feat: updated .vcf.gz output in simgenotype Dec 21, 2022
Copy link
Member

@aryarm aryarm left a comment

Choose a reason for hiding this comment

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

looks great!

in the future, I think it might be helpful to have some way to choose the file extension, so we can support bcf or plain-old vcf, but not a super huge concern right now

btw, I updated the title to include a colon after the "feat" so that we could get pass the title github action check

@mlamkin7 mlamkin7 merged commit f61f613 into main Dec 22, 2022
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.

VCF writing in simgenotype is very slow
2 participants