Skip to content

Commit 9fb8293

Browse files
committed
Ensure FASTQ file groups sorted prior to merge
1 parent bba0a70 commit 9fb8293

File tree

1 file changed

+9
-6
lines changed

1 file changed

+9
-6
lines changed

SequenceAnalysis/src/org/labkey/sequenceanalysis/FileGroup.java

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,15 @@
11
package org.labkey.sequenceanalysis;
22

3-
import org.json.JSONObject;
4-
import org.labkey.api.util.Pair;
5-
63
import java.io.File;
74
import java.io.Serializable;
85
import java.util.ArrayList;
96
import java.util.Arrays;
10-
import java.util.HashMap;
7+
import java.util.Comparator;
118
import java.util.HashSet;
129
import java.util.List;
1310
import java.util.Map;
1411
import java.util.Set;
12+
import java.util.TreeMap;
1513

1614
/**
1715
* Created by bimber on 2/19/2015.
@@ -35,7 +33,7 @@ public Set<String> getPlatformUnits()
3533
public List<List<FilePair>> groupByPlatformUnit()
3634
{
3735
List<List<FilePair>> ret = new ArrayList<>();
38-
Map<String, List<FilePair>> grouped = new HashMap<>();
36+
Map<String, List<FilePair>> grouped = new TreeMap<>();
3937

4038
for (FilePair pair : filePairs)
4139
{
@@ -47,13 +45,18 @@ public List<List<FilePair>> groupByPlatformUnit()
4745
{
4846
if (!grouped.containsKey(pair.platformUnit))
4947
{
50-
grouped.put(pair.platformUnit, new ArrayList<FilePair>());
48+
grouped.put(pair.platformUnit, new ArrayList<>());
5149
}
5250

5351
grouped.get(pair.platformUnit).add(pair);
5452
}
5553
}
5654

55+
for (String key : grouped.keySet())
56+
{
57+
grouped.get(key).sort(Comparator.comparing(filePair -> filePair.file1));
58+
}
59+
5760
ret.addAll(grouped.values());
5861

5962
return ret;

0 commit comments

Comments
 (0)