Add parameter checks to stats ops

gselzer · gselzer · commit 025dd2df43ee · 2024-01-11T10:24:26.000-06:00
diff --git a/scijava-ops-image/src/main/java/org/scijava/ops/image/stats/DefaultPercentile.java b/scijava-ops-image/src/main/java/org/scijava/ops/image/stats/DefaultPercentile.java
@@ -62,6 +62,10 @@ public class DefaultPercentile<I extends RealType<I>, O extends RealType<O>>
 	 */
 	@Override
 	public void compute(final Iterable<I> input, final Double percent, final O output) {
+		if (percent < 0 || percent > 100) {
+			throw new IllegalArgumentException(
+					"Percent must be between 0 and 100 (inclusive) but was " + percent);
+		}
 		op.compute(input, percent / 100, output);
 	}
 }
diff --git a/scijava-ops-image/src/main/java/org/scijava/ops/image/stats/DefaultQuantile.java b/scijava-ops-image/src/main/java/org/scijava/ops/image/stats/DefaultQuantile.java
@@ -64,6 +64,11 @@ public class DefaultQuantile<I extends RealType<I>, O extends RealType<O>>
 	public void compute(final Iterable<I> input, final Double quantile,
 		final O output)
 	{
+		if (quantile < 0 || quantile > 1) {
+			throw new IllegalArgumentException(
+					"Quantile must be between 0 and 1 (inclusive) but is " + quantile);
+		}
+
 		final ArrayList<Double> statistics = new ArrayList<>();
 
 		final Iterator<I> it = input.iterator();

Original file line number	Diff line number	Diff line change
`@@ -62,6 +62,10 @@ public class DefaultPercentile<I extends RealType<I>, O extends RealType<O>>`
`62`	`62`	`*/`
`63`	`63`	`@Override`
`64`	`64`	`public void compute(final Iterable<I> input, final Double percent, final O output) {`
	`65`	`+ if (percent < 0 \|\| percent > 100) {`
	`66`	`+ throw new IllegalArgumentException(`
	`67`	`+ "Percent must be between 0 and 100 (inclusive) but was " + percent);`
	`68`	`+ }`
`65`	`69`	`op.compute(input, percent / 100, output);`
`66`	`70`	`}`
`67`	`71`	`}`