per review: renamed new arguments for clarity, added a minValue for f…

…ile name number of digits
broadinstitute · Oct 6, 2021 · e65224a · e65224a
1 parent 3ea174f
commit e65224a
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/main/java/org/broadinstitute/hellbender/tools/walkers/SplitIntervals.java b/src/main/java/org/broadinstitute/hellbender/tools/walkers/SplitIntervals.java
@@ -86,14 +86,14 @@ public class SplitIntervals extends GATKTool {
 
     public static final String MIN_CONTIG_SIZE_LONG_NAME = "min-contig-size";
 
-    public static final String INTERVAL_FILE_PREFIX_FULL_NAME = "prefix";
+    public static final String INTERVAL_FILE_PREFIX_FULL_NAME = "interval-file-prefix";
     public static final String INTERVAL_FILE_EXTENSION_FULL_NAME = "extension";
 
     public static final String DEFAULT_PREFIX = "";
     public static final String PICARD_INTERVAL_FILE_EXTENSION = "interval_list";
     public static final String DEFAULT_EXTENSION = "-scattered." + PICARD_INTERVAL_FILE_EXTENSION;
 
-    public static final String INTERVAL_NUMBER_OF_DIGITS_FULL_NAME = "digits";
+    public static final String INTERVAL_NUMBER_OF_DIGITS_FULL_NAME = "interval-file-num-digits";
     public static final int DEFAULT_NUMBER_OF_DIGITS = 4;  //to preserve backward compatibility
 
     @Argument(fullName = SCATTER_COUNT_LONG_NAME, shortName = SCATTER_COUNT_SHORT_NAME,
@@ -117,7 +117,7 @@ public class SplitIntervals extends GATKTool {
     @Argument(doc = "Extension to use when writing interval files", fullName = INTERVAL_FILE_EXTENSION_FULL_NAME, optional = true)
     public String extension = DEFAULT_EXTENSION;
 
-    @Argument(doc = "Number of digits to use when writing interval files", fullName = INTERVAL_NUMBER_OF_DIGITS_FULL_NAME, optional = true)
+    @Argument(doc = "Number of digits to use when writing interval files", fullName = INTERVAL_NUMBER_OF_DIGITS_FULL_NAME, minValue = 1, optional = true)
     public int numDigits = DEFAULT_NUMBER_OF_DIGITS;
 
     @Argument(doc = "Scattered interval files do not contain intervals from multiple contigs.  This is applied after the initial scatter, so that the requested scatter count is a lower bound on the number of actual scattered files.", fullName = DONT_MIX_CONTIGS_LONG_NAME, optional = true)