fix: avoid adding duplicate CSQ/ANN fields via --retain-ann
#366
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
We have cases where we want to ensure that certain
ANN
columns are in the MAF file, and we generate that list programmatically in a workflow. Currently, this leads to a duplication of any field requested via--retain-ann
, that is already in the default list at:vcf2maf/vcf2maf.pl
Lines 545 to 552 in 0d3f514
So this solutions filters out any
--retain-add
-specified fields that are already in@ann_cols
and avoids pushing them onto the column array.I just looked up all the syntax and tried it in some online perl compiler. So feel free to adjust it to whichever style you might prefer. And obviously to fix any mistakes I might have made.