code review

hammerlab · Dec 5, 2014 · ec4ea29 · ec4ea29
1 parent 90edbc7
commit ec4ea29
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/workers/genotype_extractor.py b/workers/genotype_extractor.py
@@ -53,9 +53,8 @@ def insert_run(run, engine, connection, metadata):
     if run.get('truth_vcf_path'):
         vcfs.append({'uri': run['truth_vcf_path'], 'is_validation': True})
 
-    # do some validation involving the content of all the VCFs before modifying
-    # the database.
-    vcfs = filter(lambda v: not vcf_exists(connection, v['uri']), vcfs)
+    # Validate the contents of the VCFs before modifying the database.
+    vcfs = [v for v in vcfs if not vcf_exists(connection, v['uri'])]
     for vcf in vcfs:
         reader, header_text = load_vcf(vcf['uri'])
         if vcf['is_validation'] and len(reader.samples) > 1: