Fixed uninitialized call of variable field if VAF is None

pdxgx / neoepiscope

predicts neoepitopes from phased somatic mutations detected using tumor/normal DNA-seq data

Other

26 stars 17 forks source link

Fixed uninitialized call of variable field if VAF is None #8

Closed jafors closed 4 years ago

jafors commented 4 years ago

Hi,

I came across this little bug, if a vcf does not contain a VAF field, the newest release (0.3.6) tries to access the "field" variable without initializing it. I looked into the code and found simply returning None would be a good idea in this case (as stated in your documentation).

If you have a different fix in mind, go ahead and ignore this. ;)

maryawood commented 4 years ago

Thank you so much for pointing out this issue! There was one other part of the code related to this that I wanted to fix too, so I went ahead adjusted both myself. There is now a new release, v0.3.7, available on the release section of the repo and through PyPI.