diff --git a/public/gatk-queue-extensions-public/src/main/scala/org/broadinstitute/gatk/queue/extensions/picard/PicardBamFunction.scala b/public/gatk-queue-extensions-public/src/main/scala/org/broadinstitute/gatk/queue/extensions/picard/PicardBamFunction.scala index a2db658bc..80a60c502 100644 --- a/public/gatk-queue-extensions-public/src/main/scala/org/broadinstitute/gatk/queue/extensions/picard/PicardBamFunction.scala +++ b/public/gatk-queue-extensions-public/src/main/scala/org/broadinstitute/gatk/queue/extensions/picard/PicardBamFunction.scala @@ -27,7 +27,7 @@ package org.broadinstitute.gatk.queue.extensions.picard import java.io.File import org.broadinstitute.gatk.queue.function.JavaCommandLineFunction -import htsjdk.samtools.SAMFileReader.ValidationStringency +import htsjdk.samtools.ValidationStringency import htsjdk.samtools.SAMFileHeader.SortOrder /** diff --git a/public/gatk-queue-extensions-public/src/main/scala/org/broadinstitute/gatk/queue/extensions/picard/PicardMetricsFunction.scala b/public/gatk-queue-extensions-public/src/main/scala/org/broadinstitute/gatk/queue/extensions/picard/PicardMetricsFunction.scala index eaeaa1837..b9885a969 100644 --- a/public/gatk-queue-extensions-public/src/main/scala/org/broadinstitute/gatk/queue/extensions/picard/PicardMetricsFunction.scala +++ b/public/gatk-queue-extensions-public/src/main/scala/org/broadinstitute/gatk/queue/extensions/picard/PicardMetricsFunction.scala @@ -27,7 +27,7 @@ package org.broadinstitute.gatk.queue.extensions.picard import java.io.File import org.broadinstitute.gatk.queue.function.JavaCommandLineFunction -import htsjdk.samtools.SAMFileReader.ValidationStringency +import htsjdk.samtools.ValidationStringency import htsjdk.samtools.SAMFileHeader.SortOrder /** diff --git a/public/gatk-root/pom.xml b/public/gatk-root/pom.xml index 8d68e4cb0..af9b7ef80 100644 --- a/public/gatk-root/pom.xml +++ b/public/gatk-root/pom.xml @@ -44,8 +44,8 @@ org.testng.reporters.FailedReporter,org.testng.reporters.JUnitXMLReporter,org.broadinstitute.gatk.utils.TestNGTestTransformer,org.broadinstitute.gatk.utils.GATKTextReporter,org.uncommons.reportng.HTMLReporter - 1.112.1452 - 1.112.1452 + 1.118.1556 + 1.118.1521 diff --git a/public/gatk-tools-public/src/main/java/org/broadinstitute/gatk/engine/ReadProperties.java b/public/gatk-tools-public/src/main/java/org/broadinstitute/gatk/engine/ReadProperties.java index 86d19269a..6ee9ad3a4 100644 --- a/public/gatk-tools-public/src/main/java/org/broadinstitute/gatk/engine/ReadProperties.java +++ b/public/gatk-tools-public/src/main/java/org/broadinstitute/gatk/engine/ReadProperties.java @@ -27,6 +27,7 @@ package org.broadinstitute.gatk.engine; import htsjdk.samtools.SAMFileHeader; import htsjdk.samtools.SAMFileReader; +import htsjdk.samtools.ValidationStringency; import org.broadinstitute.gatk.engine.arguments.ValidationExclusion; import org.broadinstitute.gatk.engine.datasources.reads.SAMReaderID; import org.broadinstitute.gatk.engine.downsampling.DownsamplingMethod; @@ -56,7 +57,7 @@ public class ReadProperties { private final Collection readers; private final SAMFileHeader header; private final SAMFileHeader.SortOrder sortOrder; - private final SAMFileReader.ValidationStringency validationStringency; + private final ValidationStringency validationStringency; private final DownsamplingMethod downsamplingMethod; private final ValidationExclusion exclusionList; private final Collection supplementalFilters; @@ -107,7 +108,7 @@ public class ReadProperties { * How strict should validation be? * @return Stringency of validation. */ - public SAMFileReader.ValidationStringency getValidationStringency() { + public ValidationStringency getValidationStringency() { return validationStringency; } @@ -173,7 +174,7 @@ public class ReadProperties { SAMFileHeader header, SAMFileHeader.SortOrder sortOrder, boolean useOriginalBaseQualities, - SAMFileReader.ValidationStringency strictness, + ValidationStringency strictness, DownsamplingMethod downsamplingMethod, ValidationExclusion exclusionList, Collection supplementalFilters, diff --git a/public/gatk-tools-public/src/main/java/org/broadinstitute/gatk/engine/alignment/bwa/java/AlignerTestHarness.java b/public/gatk-tools-public/src/main/java/org/broadinstitute/gatk/engine/alignment/bwa/java/AlignerTestHarness.java index 158551ead..91e41e5b3 100644 --- a/public/gatk-tools-public/src/main/java/org/broadinstitute/gatk/engine/alignment/bwa/java/AlignerTestHarness.java +++ b/public/gatk-tools-public/src/main/java/org/broadinstitute/gatk/engine/alignment/bwa/java/AlignerTestHarness.java @@ -63,7 +63,7 @@ public class AlignerTestHarness { int count = 0; SAMFileReader reader = new SAMFileReader(bamFile); - reader.setValidationStringency(SAMFileReader.ValidationStringency.SILENT); + reader.setValidationStringency(ValidationStringency.SILENT); int mismatches = 0; int failures = 0; diff --git a/public/gatk-tools-public/src/main/java/org/broadinstitute/gatk/engine/arguments/GATKArgumentCollection.java b/public/gatk-tools-public/src/main/java/org/broadinstitute/gatk/engine/arguments/GATKArgumentCollection.java index b88db9d13..d8768ab6c 100644 --- a/public/gatk-tools-public/src/main/java/org/broadinstitute/gatk/engine/arguments/GATKArgumentCollection.java +++ b/public/gatk-tools-public/src/main/java/org/broadinstitute/gatk/engine/arguments/GATKArgumentCollection.java @@ -26,6 +26,7 @@ package org.broadinstitute.gatk.engine.arguments; import htsjdk.samtools.SAMFileReader; +import htsjdk.samtools.ValidationStringency; import org.broadinstitute.gatk.utils.commandline.*; import org.broadinstitute.gatk.engine.GenomeAnalysisEngine; import org.broadinstitute.gatk.engine.downsampling.DownsampleType; @@ -361,7 +362,7 @@ public class GATKArgumentCollection { * Keep in mind that if you set this to LENIENT, we may refuse to provide you with support if anything goes wrong. */ @Argument(fullName = "validation_strictness", shortName = "S", doc = "How strict should we be with validation", required = false) - public SAMFileReader.ValidationStringency strictnessLevel = SAMFileReader.ValidationStringency.SILENT; + public ValidationStringency strictnessLevel = ValidationStringency.SILENT; /** * Some tools keep program records in the SAM header by default. Use this argument to override that behavior and discard program records for the SAM header. */ diff --git a/public/gatk-tools-public/src/main/java/org/broadinstitute/gatk/engine/datasources/reads/SAMDataSource.java b/public/gatk-tools-public/src/main/java/org/broadinstitute/gatk/engine/datasources/reads/SAMDataSource.java index 6df0d89ba..0fc06fcce 100644 --- a/public/gatk-tools-public/src/main/java/org/broadinstitute/gatk/engine/datasources/reads/SAMDataSource.java +++ b/public/gatk-tools-public/src/main/java/org/broadinstitute/gatk/engine/datasources/reads/SAMDataSource.java @@ -86,7 +86,7 @@ public class SAMDataSource { /** * How strict are the readers driving this data source. */ - private final SAMFileReader.ValidationStringency validationStringency; + private final ValidationStringency validationStringency; /** * Do we want to remove the program records from this data source? @@ -183,7 +183,7 @@ public class SAMDataSource { numFileHandles, genomeLocParser, false, - SAMFileReader.ValidationStringency.STRICT, + ValidationStringency.STRICT, null, null, new ValidationExclusion(), @@ -202,7 +202,7 @@ public class SAMDataSource { Integer numFileHandles, GenomeLocParser genomeLocParser, boolean useOriginalBaseQualities, - SAMFileReader.ValidationStringency strictness, + ValidationStringency strictness, Integer readBufferSize, DownsamplingMethod downsamplingMethod, ValidationExclusion exclusionList, @@ -251,7 +251,7 @@ public class SAMDataSource { Integer numFileHandles, GenomeLocParser genomeLocParser, boolean useOriginalBaseQualities, - SAMFileReader.ValidationStringency strictness, + ValidationStringency strictness, Integer readBufferSize, DownsamplingMethod downsamplingMethod, ValidationExclusion exclusionList, @@ -800,7 +800,7 @@ public class SAMDataSource { * @param reader Reader for which to determine the id. * @return id of the given reader. */ - protected synchronized SAMReaderID getReaderID(SAMFileReader reader) { + protected synchronized SAMReaderID getReaderID(SamReader reader) { for(SAMReaders readers: allResources) { SAMReaderID id = readers.getReaderID(reader); if(id != null) @@ -845,7 +845,7 @@ public class SAMDataSource { * @param validationStringency validation stringency. * @param removeProgramRecords indicate whether to clear program records from the readers */ - public SAMReaders(Collection readerIDs, SAMFileReader.ValidationStringency validationStringency, boolean removeProgramRecords) { + public SAMReaders(Collection readerIDs, ValidationStringency validationStringency, boolean removeProgramRecords) { final int totalNumberOfFiles = readerIDs.size(); int readerNumber = 1; final SimpleTimer timer = new SimpleTimer().start(); @@ -1028,7 +1028,7 @@ public class SAMDataSource { * @param reader Reader for which to search. * @return The id associated the given reader, or null if the reader is not present in this collection. */ - protected SAMReaderID getReaderID(SAMFileReader reader) { + protected SAMReaderID getReaderID(SamReader reader) { for(Map.Entry entry: readers.entrySet()) { if(reader == entry.getValue()) return entry.getKey(); @@ -1132,33 +1132,11 @@ public class SAMDataSource { /** * Locates the index file alongside the given BAM, if present. - * TODO: This is currently a hachetjob that reaches into Picard and pulls out its index file locator. Replace with something more permanent. * @param bamFile The data file to use. * @return A File object if the index file is present; null otherwise. */ private File findIndexFile(File bamFile) { - File indexFile; - - try { - Class bamFileReaderClass = Class.forName("htsjdk.samtools.BAMFileReader"); - Method indexFileLocator = bamFileReaderClass.getDeclaredMethod("findIndexFile",File.class); - indexFileLocator.setAccessible(true); - indexFile = (File)indexFileLocator.invoke(null,bamFile); - } - catch(ClassNotFoundException ex) { - throw new ReviewedGATKException("Unable to locate BAMFileReader class, used to check for index files"); - } - catch(NoSuchMethodException ex) { - throw new ReviewedGATKException("Unable to locate Picard index file locator."); - } - catch(IllegalAccessException ex) { - throw new ReviewedGATKException("Unable to access Picard index file locator."); - } - catch(InvocationTargetException ex) { - throw new ReviewedGATKException("Unable to invoke Picard index file locator."); - } - - return indexFile; + return SamFiles.findIndex(bamFile); } /** diff --git a/public/gatk-tools-public/src/main/java/org/broadinstitute/gatk/engine/datasources/reads/utilities/BAMFileStat.java b/public/gatk-tools-public/src/main/java/org/broadinstitute/gatk/engine/datasources/reads/utilities/BAMFileStat.java index ca97732cf..95e0341fc 100644 --- a/public/gatk-tools-public/src/main/java/org/broadinstitute/gatk/engine/datasources/reads/utilities/BAMFileStat.java +++ b/public/gatk-tools-public/src/main/java/org/broadinstitute/gatk/engine/datasources/reads/utilities/BAMFileStat.java @@ -27,6 +27,7 @@ package org.broadinstitute.gatk.engine.datasources.reads.utilities; import htsjdk.samtools.BAMIndex; import htsjdk.samtools.SAMFileReader; +import htsjdk.samtools.ValidationStringency; import org.broadinstitute.gatk.utils.commandline.Argument; import org.broadinstitute.gatk.utils.commandline.CommandLineProgram; import org.broadinstitute.gatk.utils.exceptions.ReviewedGATKException; @@ -85,7 +86,7 @@ public class BAMFileStat extends CommandLineProgram { BAMIndex index; reader = new SAMFileReader(bamFile); - reader.setValidationStringency(SAMFileReader.ValidationStringency.SILENT); + reader.setValidationStringency(ValidationStringency.SILENT); reader.enableIndexCaching(true); index = reader.getIndex(); diff --git a/public/gatk-tools-public/src/main/java/org/broadinstitute/gatk/engine/datasources/reads/utilities/PrintBAMRegion.java b/public/gatk-tools-public/src/main/java/org/broadinstitute/gatk/engine/datasources/reads/utilities/PrintBAMRegion.java index c3edab396..b0842e161 100644 --- a/public/gatk-tools-public/src/main/java/org/broadinstitute/gatk/engine/datasources/reads/utilities/PrintBAMRegion.java +++ b/public/gatk-tools-public/src/main/java/org/broadinstitute/gatk/engine/datasources/reads/utilities/PrintBAMRegion.java @@ -25,10 +25,7 @@ package org.broadinstitute.gatk.engine.datasources.reads.utilities; -import htsjdk.samtools.GATKBAMFileSpan; -import htsjdk.samtools.GATKChunk; -import htsjdk.samtools.SAMFileReader; -import htsjdk.samtools.SAMRecordIterator; +import htsjdk.samtools.*; import org.broadinstitute.gatk.utils.commandline.Argument; import org.broadinstitute.gatk.utils.commandline.CommandLineProgram; import org.broadinstitute.gatk.utils.exceptions.UserException; @@ -58,7 +55,7 @@ public class PrintBAMRegion extends CommandLineProgram { public int execute() { SAMFileReader reader = new SAMFileReader(input); - reader.setValidationStringency(SAMFileReader.ValidationStringency.SILENT); + reader.setValidationStringency(ValidationStringency.SILENT); Pattern regionPattern = Pattern.compile("(\\d+):(\\d+)-(\\d+):(\\d+)"); Matcher matcher = regionPattern.matcher(region); diff --git a/public/gatk-tools-public/src/main/java/org/broadinstitute/gatk/engine/io/OutputTracker.java b/public/gatk-tools-public/src/main/java/org/broadinstitute/gatk/engine/io/OutputTracker.java index f32779276..8f2fbe340 100644 --- a/public/gatk-tools-public/src/main/java/org/broadinstitute/gatk/engine/io/OutputTracker.java +++ b/public/gatk-tools-public/src/main/java/org/broadinstitute/gatk/engine/io/OutputTracker.java @@ -26,6 +26,7 @@ package org.broadinstitute.gatk.engine.io; import htsjdk.samtools.SAMFileReader; +import htsjdk.samtools.ValidationStringency; import org.broadinstitute.gatk.utils.commandline.ArgumentSource; import org.broadinstitute.gatk.engine.io.storage.Storage; import org.broadinstitute.gatk.engine.io.storage.StorageFactory; @@ -77,7 +78,7 @@ public abstract class OutputTracker { */ public abstract T getStorage( Stub stub ); - public void prepareWalker( Walker walker, SAMFileReader.ValidationStringency strictnessLevel ) { + public void prepareWalker( Walker walker, ValidationStringency strictnessLevel ) { for( Map.Entry io: inputs.entrySet() ) { ArgumentSource targetField = io.getKey(); Object targetValue = io.getValue(); diff --git a/public/gatk-tools-public/src/main/java/org/broadinstitute/gatk/engine/walkers/diffengine/BAMDiffableReader.java b/public/gatk-tools-public/src/main/java/org/broadinstitute/gatk/engine/walkers/diffengine/BAMDiffableReader.java index 2c8a6c8e1..2c8cc7ae1 100644 --- a/public/gatk-tools-public/src/main/java/org/broadinstitute/gatk/engine/walkers/diffengine/BAMDiffableReader.java +++ b/public/gatk-tools-public/src/main/java/org/broadinstitute/gatk/engine/walkers/diffengine/BAMDiffableReader.java @@ -28,6 +28,7 @@ package org.broadinstitute.gatk.engine.walkers.diffengine; import htsjdk.samtools.SAMFileReader; import htsjdk.samtools.SAMRecord; import htsjdk.samtools.SAMRecordIterator; +import htsjdk.samtools.ValidationStringency; import htsjdk.samtools.util.BlockCompressedInputStream; import java.io.*; @@ -49,7 +50,7 @@ public class BAMDiffableReader implements DiffableReader { @Override public DiffElement readFromFile(File file, int maxElementsToRead) { final SAMFileReader reader = new SAMFileReader(file, null); // null because we don't want it to look for the index - reader.setValidationStringency(SAMFileReader.ValidationStringency.SILENT); + reader.setValidationStringency(ValidationStringency.SILENT); DiffNode root = DiffNode.rooted(file.getName()); SAMRecordIterator iterator = reader.iterator(); diff --git a/public/gatk-tools-public/src/main/java/org/broadinstitute/gatk/utils/sam/SAMFileReaderBuilder.java b/public/gatk-tools-public/src/main/java/org/broadinstitute/gatk/utils/sam/SAMFileReaderBuilder.java index 2c065bb24..2c5ea5f0e 100644 --- a/public/gatk-tools-public/src/main/java/org/broadinstitute/gatk/utils/sam/SAMFileReaderBuilder.java +++ b/public/gatk-tools-public/src/main/java/org/broadinstitute/gatk/utils/sam/SAMFileReaderBuilder.java @@ -26,6 +26,7 @@ package org.broadinstitute.gatk.utils.sam; import htsjdk.samtools.SAMFileReader; +import htsjdk.samtools.ValidationStringency; import org.broadinstitute.gatk.utils.exceptions.ReviewedGATKException; import java.io.File; @@ -47,7 +48,7 @@ public class SAMFileReaderBuilder { /** * What compression level should be used when building this file? */ - private SAMFileReader.ValidationStringency validationStringency = null; + private ValidationStringency validationStringency = null; /** * Sets the handle of the sam file to which data should be written. @@ -61,7 +62,7 @@ public class SAMFileReaderBuilder { * Sets the validation stringency to apply when reading this sam file. * @param validationStringency Stringency to apply. Must not be null. */ - public void setValidationStringency( SAMFileReader.ValidationStringency validationStringency ) { + public void setValidationStringency( ValidationStringency validationStringency ) { this.validationStringency = validationStringency; } diff --git a/public/gatk-tools-public/src/test/java/org/broadinstitute/gatk/engine/EngineFeaturesIntegrationTest.java b/public/gatk-tools-public/src/test/java/org/broadinstitute/gatk/engine/EngineFeaturesIntegrationTest.java index dc1b66039..bdd81e7a3 100644 --- a/public/gatk-tools-public/src/test/java/org/broadinstitute/gatk/engine/EngineFeaturesIntegrationTest.java +++ b/public/gatk-tools-public/src/test/java/org/broadinstitute/gatk/engine/EngineFeaturesIntegrationTest.java @@ -25,10 +25,7 @@ package org.broadinstitute.gatk.engine; -import htsjdk.samtools.SAMFileReader; -import htsjdk.samtools.SAMReadGroupRecord; -import htsjdk.samtools.SAMRecord; -import htsjdk.samtools.SAMSequenceDictionary; +import htsjdk.samtools.*; import htsjdk.samtools.util.CloseableIterator; import htsjdk.tribble.readers.LineIterator; import org.broadinstitute.gatk.engine.walkers.WalkerTest; @@ -308,7 +305,7 @@ public class EngineFeaturesIntegrationTest extends WalkerTest { final File outputBam = executeTest("testGATKEngineConsolidatesCigars", spec).first.get(0); final SAMFileReader reader = new SAMFileReader(outputBam); - reader.setValidationStringency(SAMFileReader.ValidationStringency.SILENT); + reader.setValidationStringency(ValidationStringency.SILENT); reader.setSAMRecordFactory(new GATKSamRecordFactory()); final SAMRecord read = reader.iterator().next(); diff --git a/public/gatk-tools-public/src/test/java/org/broadinstitute/gatk/engine/ReadMetricsUnitTest.java b/public/gatk-tools-public/src/test/java/org/broadinstitute/gatk/engine/ReadMetricsUnitTest.java index 8980d3aa6..1153bccb8 100644 --- a/public/gatk-tools-public/src/test/java/org/broadinstitute/gatk/engine/ReadMetricsUnitTest.java +++ b/public/gatk-tools-public/src/test/java/org/broadinstitute/gatk/engine/ReadMetricsUnitTest.java @@ -152,7 +152,7 @@ public class ReadMetricsUnitTest extends BaseTest { final SAMDataSource dataSource = new SAMDataSource(samFiles, new ThreadAllocation(), null, genomeLocParser, false, - SAMFileReader.ValidationStringency.STRICT, + ValidationStringency.STRICT, null, null, new ValidationExclusion(), @@ -187,7 +187,7 @@ public class ReadMetricsUnitTest extends BaseTest { final SAMDataSource dataSource = new SAMDataSource(samFiles, new ThreadAllocation(), null, genomeLocParser, false, - SAMFileReader.ValidationStringency.STRICT, + ValidationStringency.STRICT, null, null, new ValidationExclusion(), @@ -228,7 +228,7 @@ public class ReadMetricsUnitTest extends BaseTest { final SAMDataSource dataSource = new SAMDataSource(samFiles, new ThreadAllocation(), null, genomeLocParser, false, - SAMFileReader.ValidationStringency.STRICT, + ValidationStringency.STRICT, null, null, new ValidationExclusion(), @@ -275,7 +275,7 @@ public class ReadMetricsUnitTest extends BaseTest { final SAMDataSource dataSource = new SAMDataSource(samFiles, new ThreadAllocation(), null, genomeLocParser, false, - SAMFileReader.ValidationStringency.STRICT, + ValidationStringency.STRICT, null, null, new ValidationExclusion(), diff --git a/public/gatk-tools-public/src/test/java/org/broadinstitute/gatk/engine/datasources/reads/ReadShardBalancerUnitTest.java b/public/gatk-tools-public/src/test/java/org/broadinstitute/gatk/engine/datasources/reads/ReadShardBalancerUnitTest.java index 29e267693..be48194b5 100644 --- a/public/gatk-tools-public/src/test/java/org/broadinstitute/gatk/engine/datasources/reads/ReadShardBalancerUnitTest.java +++ b/public/gatk-tools-public/src/test/java/org/broadinstitute/gatk/engine/datasources/reads/ReadShardBalancerUnitTest.java @@ -88,7 +88,7 @@ public class ReadShardBalancerUnitTest extends BaseTest { null, new GenomeLocParser(header.getSequenceDictionary()), false, - SAMFileReader.ValidationStringency.SILENT, + ValidationStringency.SILENT, ReadShard.DEFAULT_MAX_READS, // reset ReadShard.MAX_READS to ReadShard.DEFAULT_MAX_READS for each test downsamplingMethod, new ValidationExclusion(), diff --git a/public/gatk-tools-public/src/test/java/org/broadinstitute/gatk/engine/datasources/reads/SAMDataSourceUnitTest.java b/public/gatk-tools-public/src/test/java/org/broadinstitute/gatk/engine/datasources/reads/SAMDataSourceUnitTest.java index e3d871707..526b8ce02 100644 --- a/public/gatk-tools-public/src/test/java/org/broadinstitute/gatk/engine/datasources/reads/SAMDataSourceUnitTest.java +++ b/public/gatk-tools-public/src/test/java/org/broadinstitute/gatk/engine/datasources/reads/SAMDataSourceUnitTest.java @@ -105,7 +105,7 @@ public class SAMDataSourceUnitTest extends BaseTest { null, genomeLocParser, false, - SAMFileReader.ValidationStringency.SILENT, + ValidationStringency.SILENT, null, null, new ValidationExclusion(), @@ -159,7 +159,7 @@ public class SAMDataSourceUnitTest extends BaseTest { null, genomeLocParser, false, - SAMFileReader.ValidationStringency.SILENT, + ValidationStringency.SILENT, null, null, new ValidationExclusion(), @@ -175,7 +175,7 @@ public class SAMDataSourceUnitTest extends BaseTest { null, genomeLocParser, false, - SAMFileReader.ValidationStringency.SILENT, + ValidationStringency.SILENT, null, null, new ValidationExclusion(), @@ -196,7 +196,7 @@ public class SAMDataSourceUnitTest extends BaseTest { null, genomeLocParser, false, - SAMFileReader.ValidationStringency.SILENT, + ValidationStringency.SILENT, null, null, new ValidationExclusion(), @@ -221,7 +221,7 @@ public class SAMDataSourceUnitTest extends BaseTest { null, genomeLocParser, false, - SAMFileReader.ValidationStringency.SILENT, + ValidationStringency.SILENT, null, null, new ValidationExclusion(), @@ -238,7 +238,7 @@ public class SAMDataSourceUnitTest extends BaseTest { null, genomeLocParser, false, - SAMFileReader.ValidationStringency.SILENT, + ValidationStringency.SILENT, null, null, new ValidationExclusion(), diff --git a/public/gatk-tools-public/src/test/java/org/broadinstitute/gatk/engine/filters/ReadFilterTest.java b/public/gatk-tools-public/src/test/java/org/broadinstitute/gatk/engine/filters/ReadFilterTest.java index 9a003d51d..f174a634b 100644 --- a/public/gatk-tools-public/src/test/java/org/broadinstitute/gatk/engine/filters/ReadFilterTest.java +++ b/public/gatk-tools-public/src/test/java/org/broadinstitute/gatk/engine/filters/ReadFilterTest.java @@ -317,7 +317,7 @@ public class ReadFilterTest extends BaseTest { final ThreadAllocation ta = new ThreadAllocation(); final Integer numFileHandles = 1; // I believe that any value would do but need to confirm. final boolean useOriginalBaseQualities = true; - final SAMFileReader.ValidationStringency strictness = SAMFileReader.ValidationStringency.LENIENT; + final ValidationStringency strictness = ValidationStringency.LENIENT; final Integer readBufferSize = 1; // not relevant. final DownsamplingMethod downsamplingMethod = DownsamplingMethod.NONE; final ValidationExclusion exclusionList = composeValidationExclusion(); diff --git a/public/gatk-tools-public/src/test/java/org/broadinstitute/gatk/engine/traversals/TraverseActiveRegionsUnitTest.java b/public/gatk-tools-public/src/test/java/org/broadinstitute/gatk/engine/traversals/TraverseActiveRegionsUnitTest.java index 918cb50dc..50eb49664 100644 --- a/public/gatk-tools-public/src/test/java/org/broadinstitute/gatk/engine/traversals/TraverseActiveRegionsUnitTest.java +++ b/public/gatk-tools-public/src/test/java/org/broadinstitute/gatk/engine/traversals/TraverseActiveRegionsUnitTest.java @@ -473,7 +473,7 @@ public class TraverseActiveRegionsUnitTest extends BaseTest { SAMDataSource dataSource = new SAMDataSource(samFiles, new ThreadAllocation(), null, genomeLocParser, false, - SAMFileReader.ValidationStringency.STRICT, + ValidationStringency.STRICT, null, null, new ValidationExclusion(), diff --git a/public/gatk-tools-public/src/test/java/org/broadinstitute/gatk/utils/locusiterator/LocusIteratorByStateBaseTest.java b/public/gatk-tools-public/src/test/java/org/broadinstitute/gatk/utils/locusiterator/LocusIteratorByStateBaseTest.java index c32325e73..286c7120c 100644 --- a/public/gatk-tools-public/src/test/java/org/broadinstitute/gatk/utils/locusiterator/LocusIteratorByStateBaseTest.java +++ b/public/gatk-tools-public/src/test/java/org/broadinstitute/gatk/utils/locusiterator/LocusIteratorByStateBaseTest.java @@ -75,7 +75,7 @@ public class LocusIteratorByStateBaseTest extends BaseTest { new SAMFileHeader(), SAMFileHeader.SortOrder.coordinate, false, - SAMFileReader.ValidationStringency.STRICT, + ValidationStringency.STRICT, downsamplingMethod, new ValidationExclusion(), Collections.emptyList(), diff --git a/public/repo/picard/picard/1.112.1452/picard-1.112.1452.jar b/public/repo/picard/picard/1.118.1521/picard-1.118.1521.jar similarity index 80% rename from public/repo/picard/picard/1.112.1452/picard-1.112.1452.jar rename to public/repo/picard/picard/1.118.1521/picard-1.118.1521.jar index c56a3a92d..f82969cca 100644 Binary files a/public/repo/picard/picard/1.112.1452/picard-1.112.1452.jar and b/public/repo/picard/picard/1.118.1521/picard-1.118.1521.jar differ diff --git a/public/repo/picard/picard/1.112.1452/picard-1.112.1452.pom b/public/repo/picard/picard/1.118.1521/picard-1.118.1521.pom similarity index 94% rename from public/repo/picard/picard/1.112.1452/picard-1.112.1452.pom rename to public/repo/picard/picard/1.118.1521/picard-1.118.1521.pom index bd617dde1..0b1f7c775 100644 --- a/public/repo/picard/picard/1.112.1452/picard-1.112.1452.pom +++ b/public/repo/picard/picard/1.118.1521/picard-1.118.1521.pom @@ -3,13 +3,13 @@ 4.0.0 picard picard - 1.112.1452 + 1.118.1521 picard samtools htsjdk - 1.112.1452 + 1.118.1556 diff --git a/public/repo/samtools/htsjdk/1.112.1452/htsjdk-1.112.1452.jar b/public/repo/samtools/htsjdk/1.118.1556/htsjdk-1.118.1556.jar similarity index 79% rename from public/repo/samtools/htsjdk/1.112.1452/htsjdk-1.112.1452.jar rename to public/repo/samtools/htsjdk/1.118.1556/htsjdk-1.118.1556.jar index 80b759ec7..d64d3ae29 100644 Binary files a/public/repo/samtools/htsjdk/1.112.1452/htsjdk-1.112.1452.jar and b/public/repo/samtools/htsjdk/1.118.1556/htsjdk-1.118.1556.jar differ diff --git a/public/repo/samtools/htsjdk/1.112.1452/htsjdk-1.112.1452.pom b/public/repo/samtools/htsjdk/1.118.1556/htsjdk-1.118.1556.pom similarity index 96% rename from public/repo/samtools/htsjdk/1.112.1452/htsjdk-1.112.1452.pom rename to public/repo/samtools/htsjdk/1.118.1556/htsjdk-1.118.1556.pom index 17cade88e..e92f8b420 100644 --- a/public/repo/samtools/htsjdk/1.112.1452/htsjdk-1.112.1452.pom +++ b/public/repo/samtools/htsjdk/1.118.1556/htsjdk-1.118.1556.pom @@ -3,7 +3,7 @@ 4.0.0 samtools htsjdk - 1.112.1452 + 1.118.1556 htsjdk