suppress SIS warning messages about unset environment variables

we don't care about it since we don't have the "proper" EPSG dataset available anyway

Author: DaMatrix

src/main/java/net/buildtheearth/terraplusplus/projection/sis/SISProjectionWrapper.java

@@ -10,14 +10,14 @@
 import net.buildtheearth.terraplusplus.control.AdvancedEarthGui;
 import net.buildtheearth.terraplusplus.projection.GeographicProjection;
 import net.buildtheearth.terraplusplus.projection.OutOfProjectionBoundsException;
+import net.buildtheearth.terraplusplus.util.compat.sis.SISHelper;
 import net.daporkchop.lib.common.function.exception.EConsumer;
 import org.apache.sis.geometry.DirectPosition2D;
 import org.apache.sis.geometry.Envelopes;
 import org.apache.sis.geometry.GeneralEnvelope;
 import org.apache.sis.internal.referencing.AxisDirections;
 import org.apache.sis.parameter.DefaultParameterValueGroup;
 import org.apache.sis.parameter.ParameterBuilder;
-import org.apache.sis.referencing.CRS;
 import org.apache.sis.referencing.operation.matrix.Matrix2;
 import org.apache.sis.referencing.operation.transform.AbstractMathTransform;
 import org.apache.sis.referencing.operation.transform.DomainDefinition;
@@ -34,7 +34,6 @@
 import org.opengis.referencing.cs.CoordinateSystemAxis;
 import org.opengis.referencing.operation.MathTransform;
 import org.opengis.referencing.operation.TransformException;
-import org.opengis.util.FactoryException;
 
 import java.text.ParseException;
 import java.util.Collection;
@@ -74,12 +73,11 @@ public SISProjectionWrapper(
                 : TPP_GEO_CRS);
     }
 
-    @SneakyThrows(FactoryException.class)
     public SISProjectionWrapper(@NonNull CoordinateReferenceSystem projectedCRS) {
         this.projectedCRS = projectedCRS;
 
-        this.toGeo = CRS.findOperation(this.projectedCRS, this.geoCRS, null).getMathTransform();
-        this.fromGeo = CRS.findOperation(this.geoCRS, this.projectedCRS, null).getMathTransform();
+        this.toGeo = SISHelper.findOperation(this.projectedCRS, this.geoCRS).getMathTransform();
+        this.fromGeo = SISHelper.findOperation(this.geoCRS, this.projectedCRS).getMathTransform();
     }
 
     @JsonGetter("standard")

src/main/java/net/buildtheearth/terraplusplus/projection/sis/WKTStandard.java

@@ -2,13 +2,20 @@
 
 import lombok.NonNull;
 import lombok.SneakyThrows;
+import net.buildtheearth.terraplusplus.util.compat.sis.SISHelper;
 import net.daporkchop.lib.common.reference.ReferenceStrength;
 import net.daporkchop.lib.common.reference.cache.Cached;
+import org.apache.sis.internal.referencing.ReferencingFactoryContainer;
 import org.apache.sis.io.wkt.Convention;
 import org.apache.sis.io.wkt.KeywordCase;
 import org.apache.sis.io.wkt.KeywordStyle;
 import org.apache.sis.io.wkt.Symbols;
 import org.apache.sis.io.wkt.WKTFormat;
+import org.opengis.referencing.crs.CRSFactory;
+import org.opengis.referencing.cs.CSFactory;
+import org.opengis.referencing.datum.DatumFactory;
+import org.opengis.referencing.operation.CoordinateOperationFactory;
+import org.opengis.referencing.operation.MathTransformFactory;
 
 import java.text.ParseException;
 import java.util.Locale;
@@ -26,18 +33,33 @@ public enum WKTStandard {
 
     static {
         {
-            WKTFormat format = new WKTFormat(Locale.ROOT, TimeZone.getDefault());
-            format.setKeywordCase(KeywordCase.UPPER_CASE);
-            format.setKeywordStyle(KeywordStyle.SHORT);
-            format.setConvention(Convention.WKT2);
-            format.setSymbols(Symbols.SQUARE_BRACKETS);
-            format.setIndentation(WKTFormat.SINGLE_LINE);
-            WKT2_2015.format = Cached.threadLocal(format::clone, ReferenceStrength.SOFT);
+            WKTFormat baseFormat = new WKTFormat(Locale.ROOT, TimeZone.getDefault());
+            baseFormat.setKeywordCase(KeywordCase.UPPER_CASE);
+            baseFormat.setKeywordStyle(KeywordStyle.SHORT);
+            baseFormat.setConvention(Convention.WKT2);
+            baseFormat.setSymbols(Symbols.SQUARE_BRACKETS);
+            baseFormat.setIndentation(WKTFormat.SINGLE_LINE);
+            WKT2_2015.setFormat(baseFormat);
         }
     }
 
     private Cached<WKTFormat> format;
 
+    private void setFormat(@NonNull WKTFormat baseFormat) {
+        this.format = Cached.threadLocal(() -> {
+            ReferencingFactoryContainer factories = SISHelper.factories();
+
+            //clone the base WKTFormat instance, and configure it to use the thread-local factory instances from SISHelper.factories()
+            WKTFormat format = baseFormat.clone();
+            format.setFactory(CRSFactory.class, factories.getCRSFactory());
+            format.setFactory(CSFactory.class, factories.getCSFactory());
+            format.setFactory(DatumFactory.class, factories.getDatumFactory());
+            format.setFactory(MathTransformFactory.class, factories.getMathTransformFactory());
+            format.setFactory(CoordinateOperationFactory.class, factories.getCoordinateOperationFactory());
+            return format;
+        }, ReferenceStrength.SOFT);
+    }
+
     public Object parse(@NonNull String wkt) throws ParseException {
         return this.format.get().parseObject(wkt);
     }

src/main/java/net/buildtheearth/terraplusplus/util/compat/sis/SISHelper.java

@@ -13,6 +13,7 @@
 import org.apache.sis.geometry.Envelopes;
 import org.apache.sis.geometry.GeneralEnvelope;
 import org.apache.sis.internal.referencing.ReferencingFactoryContainer;
+import org.apache.sis.internal.system.DataDirectory;
 import org.apache.sis.metadata.iso.citation.Citations;
 import org.apache.sis.referencing.CRS;
 import org.apache.sis.referencing.ImmutableIdentifier;
@@ -39,6 +40,11 @@
  */
 @UtilityClass
 public class SISHelper {
+    static {
+        //suppress 'The “SIS_DATA” environment variable is not set.' warnings in the log
+        DataDirectory.quiet();
+    }
+
     private static final Citation TPP_CITATION = Citations.fromName("Terra++");
 
     public static Citation tppCitation() {

src/test/java/projection/TestSISProjections.java

@@ -108,7 +108,7 @@ assert veryApproximateEquals(deriv1, deriv2, 0.01d, 2.2e-2d)
 
             deriv1 = proj1.toGeoDerivative(x, y);
             deriv2 = proj2.toGeoDerivative(x, y);
-            assert true || veryApproximateEquals(deriv1, deriv2, 0.01d, 1e-1d)
+            assert veryApproximateEquals(deriv1, deriv2, 0.01d, 1e-1d)
                     : "toGeoDerivative #" + i + " (" + lat + "°N, " + lon + "°E) -> (" + x + ", " + y + "):\n" + deriv1 + "!=\n" + deriv2;
         }
     }