This is an automated email from the ASF dual-hosted git repository.

aherbert pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-statistics.git

commit e17836cb1dbae0855691e568b3256f0a1a696a92
Author: Alex Herbert <aherb...@apache.org>
AuthorDate: Mon Mar 24 15:54:32 2025 +0000

    Sonar fix: remove use of restricted keyword var
---
 .../distribution/NakagamiDistributionTest.java         |  2 +-
 .../distribution/TrapezoidalDistributionTest.java      |  4 ++--
 .../distribution/TruncatedNormalDistributionTest.java  | 18 +++++++++---------
 .../commons/statistics/distribution/UserGuideTest.java |  4 ++--
 4 files changed, 14 insertions(+), 14 deletions(-)

diff --git 
a/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/NakagamiDistributionTest.java
 
b/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/NakagamiDistributionTest.java
index 711cf68..bc1b2ae 100644
--- 
a/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/NakagamiDistributionTest.java
+++ 
b/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/NakagamiDistributionTest.java
@@ -101,7 +101,7 @@ class NakagamiDistributionTest extends 
BaseContinuousDistributionTest {
         // #define quad boost::multiprecision::float128
         // T v = boost::math::tgamma_delta_ratio(mu, T(0.5));
         // T mean = sqrt(omega / mu) / v;
-        // T var = omega - (omega / mu) / v / v;
+        // T variance = omega - (omega / mu) / v / v;
         "175, 0.75, 0.865407035923572335404337637742305354, 
0.00107066217397678136642741884083229635",
         "175, 1, 0.999285970298141244170512691211913862, 
0.0014275495653023751552365584544430618",
         "175, 1.25, 1.11723567927423980521693795242933784, 
0.00178443695662796894404569806805382725",
diff --git 
a/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/TrapezoidalDistributionTest.java
 
b/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/TrapezoidalDistributionTest.java
index 0d0915c..361a6a8 100644
--- 
a/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/TrapezoidalDistributionTest.java
+++ 
b/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/TrapezoidalDistributionTest.java
@@ -128,8 +128,8 @@ class TrapezoidalDistributionTest extends 
BaseContinuousDistributionTest {
         divisor = divisor.multiply(BigDecimal.valueOf(2));
         dc = dd.pow(4).subtract(cc.pow(4)).divide(dd.subtract(cc), mc);
         ba = bb.pow(4).subtract(aa.pow(4)).divide(bb.subtract(aa), mc);
-        final BigDecimal var = dc.subtract(ba).divide(divisor, 
mc).subtract(mu.pow(2));
-        return Arguments.of(a, b, c, d, mu.doubleValue(), var.doubleValue(), 
ulps);
+        final BigDecimal variance = dc.subtract(ba).divide(divisor, 
mc).subtract(mu.pow(2));
+        return Arguments.of(a, b, c, d, mu.doubleValue(), 
variance.doubleValue(), ulps);
     }
 
     /**
diff --git 
a/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/TruncatedNormalDistributionTest.java
 
b/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/TruncatedNormalDistributionTest.java
index 50d55de..919654e 100644
--- 
a/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/TruncatedNormalDistributionTest.java
+++ 
b/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/TruncatedNormalDistributionTest.java
@@ -340,10 +340,10 @@ class TruncatedNormalDistributionTest extends 
BaseContinuousDistributionTest {
             // Allow small absolute variances using an absolute threshold of
             // machine epsilon (2^-52) * 1.5. Any true variance approaching 
machine epsilon
             // is allowed to be computed as small or zero but cannot be too 
large.
-            final double var = TruncatedNormalDistribution.variance(lower, 
upper);
-            Assertions.assertTrue(var >= 0, () -> "Variance is not positive: " 
+ var);
-            Assertions.assertEquals(var, 
TruncatedNormalDistribution.variance(-upper, -lower));
-            TestUtils.assertEquals(variance, var,
+            final double v = TruncatedNormalDistribution.variance(lower, 
upper);
+            Assertions.assertTrue(v >= 0, () -> "Variance is not positive: " + 
v);
+            Assertions.assertEquals(v, 
TruncatedNormalDistribution.variance(-upper, -lower));
+            TestUtils.assertEquals(variance, v,
                     createAbsOrRelTolerance(1.5 * 0x1.0p-52, 
-varianceRelativeError),
                 () -> String.format("variance(%s, %s)", lower, upper));
         } else {
@@ -378,9 +378,9 @@ class TruncatedNormalDistributionTest extends 
BaseContinuousDistributionTest {
      * Assert the variance of the truncated normal distribution is within the 
provided relative error.
      */
     private static void assertVariance(double lower, double upper, double 
expected, double eps) {
-        final double var = TruncatedNormalDistribution.variance(lower, upper);
-        Assertions.assertEquals(var, 
TruncatedNormalDistribution.variance(-upper, -lower));
-        TestUtils.assertEquals(expected, var, DoubleTolerances.relative(eps),
+        final double variance = TruncatedNormalDistribution.variance(lower, 
upper);
+        Assertions.assertEquals(variance, 
TruncatedNormalDistribution.variance(-upper, -lower));
+        TestUtils.assertEquals(expected, variance, 
DoubleTolerances.relative(eps),
             () -> String.format("variance(%s, %s)", lower, upper));
     }
 
@@ -392,8 +392,8 @@ class TruncatedNormalDistributionTest extends 
BaseContinuousDistributionTest {
     private static void assertVariance(double lower, double upper, double u, 
double s, double expected, double eps) {
         final double a = (lower - u) / s;
         final double b = (upper - u) / s;
-        final double var = TruncatedNormalDistribution.variance(a, b) * s * s;
-        TestUtils.assertEquals(expected, var, DoubleTolerances.relative(eps),
+        final double variance = TruncatedNormalDistribution.variance(a, b) * s 
* s;
+        TestUtils.assertEquals(expected, variance, 
DoubleTolerances.relative(eps),
             () -> String.format("variance(%s, %s, %s, %s)", lower, upper, u, 
s));
     }
 }
diff --git 
a/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/UserGuideTest.java
 
b/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/UserGuideTest.java
index 6e0ac77..c1bbb98 100644
--- 
a/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/UserGuideTest.java
+++ 
b/commons-statistics-distribution/src/test/java/org/apache/commons/statistics/distribution/UserGuideTest.java
@@ -63,7 +63,7 @@ class UserGuideTest {
         ChiSquaredDistribution chi2 = ChiSquaredDistribution.of(42);
         double df = chi2.getDegreesOfFreedom();    // 42
         double mean = chi2.getMean();              // 42
-        double var = chi2.getVariance();           // 84
+        double variance = chi2.getVariance();           // 84
 
         CauchyDistribution cauchy = CauchyDistribution.of(1.23, 4.56);
         double location = cauchy.getLocation();    // 1.23
@@ -73,7 +73,7 @@ class UserGuideTest {
 
         Assertions.assertEquals(42, df);
         Assertions.assertEquals(42, mean);
-        Assertions.assertEquals(84, var);
+        Assertions.assertEquals(84, variance);
         Assertions.assertEquals(1.23, location);
         Assertions.assertEquals(4.56, scale);
         Assertions.assertEquals(Double.NaN, undefined1);

Reply via email to