More testing.

Makefile

@@ -0,0 +1,6 @@
+
+
+
+.PHONY: act
+act:
+	act --job build

README.adoc

@@ -435,12 +435,9 @@ Like this it can be consulted
 .Accessing configuration
 [source,java]
 ----
-import static org.meeuw.configuration.*;
-import static org.meeuw.math.text.spi.FormatService.*;
-
- Configuration configuration = ConfigurationService.getConfiguration();
- NumberConfiguration aspect = configuration.getAspect(NumberConfiguration.class);
- int minimalExponent = aspect.getMinimalExponent();
+include::mihxil-math/src/test/java/org/meeuw/test/examples/ConfigurationExample.java[tag=import]
+...
+include::mihxil-math/src/test/java/org/meeuw/test/examples/ConfigurationExample.java[tag=access, indent=0]
 ----
 This would however probably mainly be used in _implementations_.
 
@@ -452,12 +449,7 @@ Actual configuration can be done in two basically distinct ways.
 .temporary overrides
 [source,java]
 ----
-Configuration configuration = ConfigurationService.getConfiguration();
-ConfiguraationService.setConfiguration(configuration.toBuilder().aspect(NumberConfiguration.class, (nc) -> nc.withMinimalExponent(8)).build());
-
-... code ...
-
-ConfigurationService.resetToDefaults();
+include::mihxil-math/src/test/java/org/meeuw/test/examples/ConfigurationExample.java[tag=configurationService,indent=-]
 ----
 
 There are some utilities in `ConfigurationService` that makes this process a bit easier.

TODO.adoc

@@ -0,0 +1,3 @@
+= todo
+
+-

mihxil-configuration/src/main/java/org/meeuw/configuration/ConfigurationService.java

@@ -91,6 +91,29 @@ public  static void  setConfiguration(Configuration configuration) {
         CONFIGURATION.set(configuration);
     }
 
+    /**
+     * @since 0.10
+     */
+    public  static void  setConfiguration(Configuration configuration, Consumer<Configuration.Builder> consumer) {
+        Configuration.Builder builder = configuration.toBuilder();
+        consumer.accept(builder);
+        setConfiguration(builder);
+    }
+    /**
+     * @since 0.10
+     */
+    public  static void  setConfiguration(Consumer<Configuration.Builder> consumer) {
+        setConfiguration(getConfiguration(), consumer);
+    }
+
+    /**
+     *
+     * @since 0.10
+     */
+    public  static void  setConfiguration(Configuration.Builder configuration) {
+        setConfiguration(configuration.build());
+    }
+
     /**
      * Unsets the configuration thread local, effectively resetting it the default settings.
      */

mihxil-configuration/src/test/java/org/meeuw/test/configuration/ConfigurationServiceTest.java

@@ -64,9 +64,8 @@ public void getAndSetConfiguration() {
         TestConfigurationAspect aspect = configuration.getAspect(TestConfigurationAspect.class);
         int someInt = aspect.getSomeInt();
         log.info(() -> String.format("some int: %d", someInt));
-        ConfigurationService.setConfiguration(configuration.toBuilder()
-            .configure(TestConfigurationAspect.class, (nc) -> nc.withSomeInt(8))
-            .build()
+        ConfigurationService.setConfiguration(builder ->
+            builder.configure(TestConfigurationAspect.class, (nc) -> nc.withSomeInt(8))
         );
         assertThat(getConfiguration()
             .getAspectValue(TestConfigurationAspect.class, TestConfigurationAspect::getSomeInt)

mihxil-math-theories/src/main/java/org/meeuw/math/abstractalgebra/test/MetricSpaceTheory.java

@@ -48,7 +48,7 @@ default void distancePositive(@ForAll(ELEMENTS) E a, @ForAll(ELEMENTS) E b) {
     }
 
     @Property
-    default void identifyOfIndiscernibles(@ForAll(ELEMENTS) E a) {
+    default void identityOfIndiscernibles(@ForAll(ELEMENTS) E a) {
         assertThat(a.distanceTo(a).isZero()).isTrue();
     }
 

mihxil-math/src/test/java/org/meeuw/test/examples/ConfigurationExample.java

@@ -0,0 +1,31 @@
+package org.meeuw.test.examples;
+// tag::import[]
+
+import org.meeuw.configuration.Configuration;
+import org.meeuw.configuration.ConfigurationService;
+import org.meeuw.math.text.configuration.NumberConfiguration;
+// end::import[]
+public class ConfigurationExample {
+
+    public static void access() {
+        // tag::access[]
+
+        Configuration configuration = ConfigurationService.getConfiguration();
+        NumberConfiguration aspect = configuration.getAspect(NumberConfiguration.class);
+        int minimalExponent = aspect.getMinimalExponent();
+        // end::access[]
+    }
+
+    public static void setConfiguration(String[] argv) {
+        // tag::configurationService[]
+        ConfigurationService.setConfiguration(builder ->
+            builder.configure(NumberConfiguration.class,
+                (numberConfiguration) -> numberConfiguration.withMinimalExponent(8)
+            )
+        );
+
+        //...code...
+        ConfigurationService.resetToDefaults();
+        // end::configurationService[]
+    }
+}

mihxil-physics/src/main/java/org/meeuw/physics/PhysicalNumber.java

@@ -179,6 +179,9 @@ public PhysicalNumber toUnits(Unit... units) {
         return toUnits(Units.of(units));
     }
 
+    /**
+     * Converts the current physical number to a different {@link SystemOfMeasurements}.
+     */
     public PhysicalNumber toUnits(SystemOfMeasurements systemOfMeasurements) {
         return toUnits(systemOfMeasurements.forDimensions(units.getDimensions()));
     }

mihxil-physics/src/main/java/org/meeuw/physics/Prefix.java

@@ -21,7 +21,7 @@
 import java.util.function.Supplier;
 
 /**
- * Multiplication prefix that can be used on a {@link Unit}, to get a new {@code code} that differs a certain factor only, but has the same {@link DimensionalAnalysis}
+ * Multiplication prefix that can be used on a {@link Unit}, to get a new {@code Unit} that differs a certain factor only, but has the same {@link DimensionalAnalysis}
  *
  * @author Michiel Meeuwissen
  * @since 0.4

mihxil-physics/src/main/java/org/meeuw/physics/PrefixedUnit.java

@@ -22,6 +22,9 @@
 
 import org.meeuw.math.uncertainnumbers.field.UncertainReal;
 
+/**
+ * A unit wrapping another {@link Unit} with a {@link Prefix}, and it just a multiple of that unit.
+ */
 public class PrefixedUnit implements Unit {
 
     private final Unit wrapped;

mihxil-physics/src/main/java/org/meeuw/physics/SystemOfMeasurements.java

@@ -23,6 +23,9 @@
 import org.meeuw.math.uncertainnumbers.field.UncertainRealField;
 
 /**
+ * There are different 'systems of measurements' in use. E.g. {@link SI}  and {@link Planck}.
+ * <p>
+ *
  * @author Michiel Meeuwissen
  * @since 0.6
  */
@@ -32,7 +35,8 @@ public interface SystemOfMeasurements {
     Unit forDimension(Dimension dimension);
 
     /**
-     * Returns in this system of measurements the preferred units for the given dimensional analysis
+     * Returns in this system of measurements the preferred units for the given dimensional analysis. This may not
+     * always be unique.
      */
     @NonNull
     default Units forDimensions(DimensionalAnalysis dimensionalAnalysis) {
@@ -44,6 +48,14 @@ default Units forDimensions(DimensionalAnalysis dimensionalAnalysis) {
             .reduce(UncertainRealField.INSTANCE.one(), UncertainReal::times);
         return new CompositeUnits(siFactor, unitExponents);
     }
+    default Units forQuantity(Quantity quantity) {
+        for (Units unit : getUnits()) {
+            if (unit.getQuantities().contains(quantity)) {
+                return unit;
+            }
+        }
+        return forDimensions(quantity.getDimensionalAnalysis());
+    }
 
 
     default List<BaseUnit> getBaseUnits() {
@@ -58,14 +70,7 @@ default List<Units> getUnits()  {
         return Collections.unmodifiableList(result);
     }
 
-    default Units forQuantity(Quantity quantity) {
-        for (Units unit : getUnits()) {
-            if (unit.getQuantities().contains(quantity)) {
-                return unit;
-            }
-        }
-        return forDimensions(quantity.getDimensionalAnalysis());
-    }
+
 
     default Units forDimensions(DimensionExponent... dimensions) {
         return forDimensions(DimensionalAnalysis.of(dimensions));

mihxil-physics/src/main/java/org/meeuw/physics/Units.java

@@ -35,6 +35,9 @@ public interface Units extends
 
     Units DIMENSIONLESS = of(exactly(1));
 
+    /**
+     * The physical units form a {@link org.meeuw.math.abstractalgebra.MultiplicativeGroup}, implemented by {@link UnitsGroup}.
+     */
     @Override
     default UnitsGroup getStructure() {
         return UnitsGroup.INSTANCE;

mihxil-physics/src/main/java/org/meeuw/physics/UnitsGroup.java

@@ -24,6 +24,9 @@
 import static org.meeuw.math.uncertainnumbers.field.UncertainDoubleElement.exactly;
 
 /**
+ * The {@link MultiplicativeAbelianGroup} of {@link Units}.
+ * <p>
+ *
  * @author Michiel Meeuwissen
  * @since 0.4
  */
@@ -70,8 +73,9 @@ public Stream<Units> stream() {
     @Override
     public Units nextRandom(Random random) {
         UnitExponent[] units = new UnitExponent[SIUnit.values().length];
+
         for (int i = 0; i < units.length; i++) {
-            units[i] = new UnitExponent(SIUnit.values()[i], random.nextInt(20));
+            units[i] = new UnitExponent(SIUnit.values()[i],  (int) (random.nextGaussian() * 3));
         }
         return new CompositeUnits(exactly(1), units);
 

mihxil-physics/src/test/java/org/meeuw/physics/PlanckTest.java

@@ -31,13 +31,11 @@ public void c() {
         assertThat(cSI.toString()).isEqualTo("(2.99792 ± 0.00005)·10⁸ m·s⁻¹");
 
         assertThat(cSI.toUnits(Planck.INSTANCE).toString()).isEqualTo("1.00000 ± 0.00002 ℓₚ·tₚ⁻¹");
-
     }
 
     @Test
-    public void kB() {
+    public void BoltzmannConstant() {
         assertThat(Planck.kB.toString()).isEqualTo("1 ℓₚ·tₚ⁻¹");
-
     }
 
 }