From 66aee613e250c21172a5a60b9f31cf8946cbab3e Mon Sep 17 00:00:00 2001 From: Mark DePristo Date: Sun, 8 Jul 2012 11:41:07 -0400 Subject: [PATCH] Bugfix for set key in mergeInfoWithMaxAC. -- Previous version was always setting set=source of info with highest AC. Should actually have been set to the set annotation value itself. --- .../sting/utils/variantcontext/VariantContextUtils.java | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/public/java/src/org/broadinstitute/sting/utils/variantcontext/VariantContextUtils.java b/public/java/src/org/broadinstitute/sting/utils/variantcontext/VariantContextUtils.java index 843838972..d7e072980 100755 --- a/public/java/src/org/broadinstitute/sting/utils/variantcontext/VariantContextUtils.java +++ b/public/java/src/org/broadinstitute/sting/utils/variantcontext/VariantContextUtils.java @@ -644,7 +644,9 @@ public class VariantContextUtils { if ( setKey != null ) { attributes.put(setKey, setValue); - if( mergeInfoWithMaxAC && vcWithMaxAC != null ) { attributesWithMaxAC.put(setKey, vcWithMaxAC.getSource()); } + if( mergeInfoWithMaxAC && vcWithMaxAC != null ) { + attributesWithMaxAC.put(setKey, setValue); + } } }