-
Notifications
You must be signed in to change notification settings - Fork 596
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Remove hardcoded limit on max homopolymer call (#8088)
* Fixing hard coded max hmer in flow matrix mods * Added comment regarding MAX_CLASS * Make code determine the collapsing automatically * Fixed a bug in assembly complexity annotation that caused difference in results between JAVA8 and JAVA11
- Loading branch information
1 parent
894117c
commit 84ade40
Showing
14 changed files
with
72 additions
and
15 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
32 changes: 32 additions & 0 deletions
32
src/test/java/org/broadinstitute/hellbender/utils/read/FlowBasedReadUtilsUnitTest.java
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,32 @@ | ||
package org.broadinstitute.hellbender.utils.read; | ||
import htsjdk.samtools.SAMFileHeader; | ||
import htsjdk.samtools.SAMReadGroupRecord; | ||
import htsjdk.samtools.SamReader; | ||
import htsjdk.samtools.SamReaderFactory; | ||
import org.broadinstitute.hellbender.GATKBaseTest; | ||
import org.testng.annotations.Test; | ||
|
||
import java.io.File; | ||
import java.nio.file.FileSystems; | ||
import java.nio.file.Path; | ||
|
||
|
||
public class FlowBasedReadUtilsUnitTest extends GATKBaseTest{ | ||
@Test | ||
void testReadGroupParsing() throws Exception{ | ||
final String testResourceDir = publicTestDir + "org/broadinstitute/hellbender/utils/read/flow/reads/"; | ||
final String inputDir = testResourceDir + "/input/"; | ||
|
||
final Path inputFile = FileSystems.getDefault().getPath(inputDir, "sample_mc.bam"); | ||
final SamReader reader = SamReaderFactory.makeDefault().open(new File(inputFile.toString())); | ||
SAMFileHeader header = reader.getFileHeader(); | ||
SAMReadGroupRecord rg1 = header.getReadGroup("UGAv3-72"); | ||
FlowBasedReadUtils.ReadGroupInfo frg1 = new FlowBasedReadUtils.ReadGroupInfo(rg1); | ||
assert(frg1.maxClass==12); | ||
assert(frg1.flowOrder.startsWith("TGCA")); | ||
SAMReadGroupRecord rg2 = header.getReadGroup("UGAv3-73"); | ||
FlowBasedReadUtils.ReadGroupInfo frg2 = new FlowBasedReadUtils.ReadGroupInfo(rg2); | ||
assert(frg2.maxClass==20); | ||
assert(frg2.flowOrder.startsWith("TGCA")); | ||
} | ||
} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Binary file modified
BIN
+8 Bytes
(100%)
...haplotypecaller/testGvcfWithAssemblyComplexityAnnotationRevamp.expected.flowbased.vcf.idx
Binary file not shown.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Binary file modified
BIN
+8 Bytes
(100%)
.../tools/haplotypecaller/testVcfWithAssemblyComplexityAnnotation.expected.flowbased.vcf.idx
Binary file not shown.
Binary file added
BIN
+115 KB
src/test/resources/org/broadinstitute/hellbender/utils/read/flow/reads/input/sample_mc.bam
Binary file not shown.