Skip to content

Don't crash on an empty vcf#200

Open
kdm9 wants to merge 1 commit intoryanlayer:masterfrom
gekkonid:empty_vcf_crash
Open

Don't crash on an empty vcf#200
kdm9 wants to merge 1 commit intoryanlayer:masterfrom
gekkonid:empty_vcf_crash

Conversation

@kdm9
Copy link
Contributor

@kdm9 kdm9 commented Jul 13, 2024

Hi folks,

In samplot vcf, the accumulator variable for the number of parsed variants uses enumerate(), which means that the var_count variable wasn't define if there were no variants in the file. Likely, an empty vcf should be an error that is reported somewhere, but for now I at least define the variable so as not to get a rather inscrutable NameError.

Thanks
K

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.

1 participant