Remove extraneous logging, disable variant explorer related unit test

hms-dbmi · Apr 2, 2024 · 4bd5787 · 4bd5787
1 parent 887dad2
commit 4bd5787
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 14 deletions.
diff --git a/etl/src/main/java/edu/harvard/hms/dbmi/avillach/hpds/etl/genotype/GenomicDatasetMerger.java b/etl/src/main/java/edu/harvard/hms/dbmi/avillach/hpds/etl/genotype/GenomicDatasetMerger.java
@@ -246,20 +246,6 @@ public FileBackedJsonIndexStorage<Integer, ConcurrentHashMap<String, VariableVar
                 }
 
                 VariableVariantMasks mergeResult = VariableVariantMasks.append(entry.getValue(), variantStore1PatientCount, variantMasks2, variantStore2PatientCount);
-                if (entry.getKey().equals("chr3,11805,T,A,LINC01986,splice_donor_region_variant")) {
-                    log.info("Mask 1: " + Optional.ofNullable(entry.getValue().heterozygousMask).map(VariantMask::toString).orElse("empty"));
-                    log.info("Mask 2: " + Optional.ofNullable(variantMasks2.heterozygousMask).map(VariantMask::toString).orElse("empty"));
-                    log.info("Merged mask: " + Optional.ofNullable(mergeResult.heterozygousMask).map(VariantMask::toString).orElse("empty"));
-
-                    Set<Integer> patientIds1 = Optional.ofNullable(entry.getValue().heterozygousMask).orElse(VariantMask.emptyInstance()).patientMaskToPatientIdSet(List.of(variantStore1.getPatientIds()));
-                    log.info("Patients 1: " + Joiner.on(",").join(patientIds1));
-
-                    Set<Integer> patientIds2 = Optional.ofNullable(variantMasks2.heterozygousMask).orElse(VariantMask.emptyInstance()).patientMaskToPatientIdSet(List.of(variantStore2.getPatientIds()));
-                    log.info("Patients 2: " + Joiner.on(",").join(patientIds2));
-
-                    Set<Integer> mergedPatientIds = Optional.ofNullable(mergeResult.heterozygousMask).orElse(VariantMask.emptyInstance()).patientMaskToPatientIdSet(List.of(mergedVariantStore.getPatientIds()));
-                    log.info("Merged patient ids: " + Joiner.on(",").join(mergedPatientIds));
-                }
                 mergedMasks.put(entry.getKey(), mergeResult);
             }
             // Any entry in the second set that is not in the merged set can be merged with an empty variant mask,

diff --git a/.../test/java/edu/harvard/hms/dbmi/avillach/hpds/data/genotype/VariantMetadataIndexTest.java b/.../test/java/edu/harvard/hms/dbmi/avillach/hpds/data/genotype/VariantMetadataIndexTest.java
@@ -9,13 +9,16 @@
 import edu.harvard.hms.dbmi.avillach.hpds.data.genotype.caching.VariantBucketHolder;
 import edu.harvard.hms.dbmi.avillach.hpds.etl.genotype.VariantMetadataLoader;
 import org.junit.jupiter.api.BeforeAll;
+import org.junit.jupiter.api.Disabled;
 import org.junit.jupiter.api.Test;
 import org.springframework.test.context.event.annotation.BeforeTestClass;
 
 import static org.junit.jupiter.api.Assertions.assertThrows;
 import static org.springframework.test.util.AssertionErrors.assertEquals;
 import static org.springframework.test.util.AssertionErrors.assertNotNull;
 
+// todo: enable when variant explorer is implemented
+@Disabled
 public class VariantMetadataIndexTest {
 
 	//From file 1 14	19038291	rs550062154	C	A