Skip to content

Commit eba5e59

Browse files
committed
Enforce consistent order of pangolin lineage calls
1 parent 86d520a commit eba5e59

File tree

1 file changed

+7
-6
lines changed

1 file changed

+7
-6
lines changed

SequenceAnalysis/src/org/labkey/sequenceanalysis/run/analysis/PangolinHandler.java

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -48,6 +48,7 @@
4848
import java.util.Collections;
4949
import java.util.HashMap;
5050
import java.util.HashSet;
51+
import java.util.LinkedHashSet;
5152
import java.util.List;
5253
import java.util.Map;
5354
import java.util.Set;
@@ -239,7 +240,7 @@ public void processFilesRemote(List<SequenceOutputFile> inputFiles, JobContext c
239240
vals.add(String.valueOf(so.getRowid()));
240241
for (String key : PANGO_FIELDS)
241242
{
242-
vals.add(pangolinData.containsKey(key) ? pangolinData.get(key) : "");
243+
vals.add(pangolinData.getOrDefault(key, ""));
243244
}
244245

245246
writer.writeNext(vals.toArray(new String[0]));
@@ -382,25 +383,25 @@ public static Map<String, String> runPangolin(File workDir, File consensusFasta,
382383
{
383384
Set<String> vals;
384385

385-
vals = ret.containsKey("PangolinLineage") ? ret.get("PangolinLineage") : new HashSet<>();
386+
vals = ret.containsKey("PangolinLineage") ? ret.get("PangolinLineage") : new LinkedHashSet<>();
386387
vals.add(pangolinData[1]);
387388
ret.put("PangolinLineage", vals);
388389

389-
vals = ret.containsKey("PangolinConflicts") ? ret.get("PangolinConflicts") : new HashSet<>();
390+
vals = ret.containsKey("PangolinConflicts") ? ret.get("PangolinConflicts") : new LinkedHashSet<>();
390391
vals.add(pangolinData[2]);
391392
ret.put("PangolinConflicts", vals);
392393

393-
vals = ret.containsKey("PangolinAmbiguity") ? ret.get("PangolinAmbiguity") : new HashSet<>();
394+
vals = ret.containsKey("PangolinAmbiguity") ? ret.get("PangolinAmbiguity") : new LinkedHashSet<>();
394395
vals.add(pangolinData[3]);
395396
ret.put("PangolinAmbiguity", vals);
396397

397-
vals = ret.containsKey("PangolinVersions") ? ret.get("PangolinVersions") : new HashSet<>();
398+
vals = ret.containsKey("PangolinVersions") ? ret.get("PangolinVersions") : new LinkedHashSet<>();
398399
vals.add(StringUtils.join(versions, ","));
399400
ret.put("PangolinVersions", vals);
400401
}
401402
else
402403
{
403-
Set<String> vals = ret.containsKey("PangolinLineage") ? ret.get("PangolinLineage") : new HashSet<>();
404+
Set<String> vals = ret.containsKey("PangolinLineage") ? ret.get("PangolinLineage") : new LinkedHashSet<>();
404405
vals.add("QC Fail");
405406
ret.put("PangolinLineage", vals);
406407
}

0 commit comments

Comments
 (0)