Default EnvironmentProbe SH baking to FAST and remove AUTO mode (#2817)

* Initial plan

* Default env baker SH mode to FAST and remove AUTO path

Agent-Logs-Url: https://github.com/jMonkeyEngine/jmonkeyengine/sessions/e70ed475-af44-4c2b-9f29-f76d9a0422b8

Co-authored-by: riccardobl <4943530+riccardobl@users.noreply.github.com>

* Harden SH mode deserialization compatibility handling

Agent-Logs-Url: https://github.com/jMonkeyEngine/jmonkeyengine/sessions/e70ed475-af44-4c2b-9f29-f76d9a0422b8

Co-authored-by: riccardobl <4943530+riccardobl@users.noreply.github.com>

* Simplify legacy SH mode mapping fallback

Agent-Logs-Url: https://github.com/jMonkeyEngine/jmonkeyengine/sessions/e70ed475-af44-4c2b-9f29-f76d9a0422b8

Co-authored-by: riccardobl <4943530+riccardobl@users.noreply.github.com>

* Avoid NPE catch in SH mode fallback parsing

Agent-Logs-Url: https://github.com/jMonkeyEngine/jmonkeyengine/sessions/e70ed475-af44-4c2b-9f29-f76d9a0422b8

Co-authored-by: riccardobl <4943530+riccardobl@users.noreply.github.com>

* Remove EnvironmentProbeControlTest per PR feedback

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: riccardobl <4943530+riccardobl@users.noreply.github.com>

Author: Copilot

jme3-core/src/main/java/com/jme3/environment/EnvironmentProbeControl.java

@@ -86,7 +86,7 @@ public class EnvironmentProbeControl extends LightProbe implements Control {
     private String uuid = "none";
     private boolean enabled = true;
     private IBLGLEnvBakerLight.SphericalHarmonicsMode sphericalHarmonicsMode =
-            IBLGLEnvBakerLight.SphericalHarmonicsMode.AUTO;
+            IBLGLEnvBakerLight.SphericalHarmonicsMode.FAST;
     private Predicate<Geometry> filter = (s) -> {
         return s.getUserData("tags.env") != null || s.getUserData("tags.env.env" + uuid) != null;
     };
@@ -371,6 +371,18 @@ public Spatial getSpatial() {
         return spatial;
     }
 
+    static IBLGLEnvBakerLight.SphericalHarmonicsMode readSphericalHarmonicsMode(String modeName) {
+        if (modeName == null) {
+            return IBLGLEnvBakerLight.SphericalHarmonicsMode.FAST;
+        }
+        try {
+            return IBLGLEnvBakerLight.SphericalHarmonicsMode.valueOf(modeName);
+        } catch (IllegalArgumentException ex) {
+            // Legacy AUTO and unknown values now fall back to FAST.
+            return IBLGLEnvBakerLight.SphericalHarmonicsMode.FAST;
+        }
+    }
+
     @Override
     public void write(JmeExporter ex) throws IOException {
         super.write(ex);
@@ -384,7 +396,7 @@ public void write(JmeExporter ex) throws IOException {
         oc.write(frustumNear, "frustumNear", 0.001f);
         oc.write(uuid, "envProbeControlUUID", "none");
         oc.write(sphericalHarmonicsMode, "sphericalHarmonicsMode",
-                IBLGLEnvBakerLight.SphericalHarmonicsMode.AUTO);
+                IBLGLEnvBakerLight.SphericalHarmonicsMode.FAST);
     }
 
     @Override
@@ -400,9 +412,15 @@ public void read(JmeImporter im) throws IOException {
         frustumFar = ic.readFloat("frustumFar", 1000f);
         frustumNear = ic.readFloat("frustumNear", 0.001f);
         uuid = ic.readString("envProbeControlUUID", "none");
-        sphericalHarmonicsMode = ic.readEnum("sphericalHarmonicsMode",
-                IBLGLEnvBakerLight.SphericalHarmonicsMode.class,
-                IBLGLEnvBakerLight.SphericalHarmonicsMode.AUTO);
+        try {
+            sphericalHarmonicsMode = ic.readEnum("sphericalHarmonicsMode",
+                    IBLGLEnvBakerLight.SphericalHarmonicsMode.class,
+                    IBLGLEnvBakerLight.SphericalHarmonicsMode.FAST);
+        } catch (IllegalArgumentException ex) {
+            sphericalHarmonicsMode = readSphericalHarmonicsMode(
+                    ic.readString("sphericalHarmonicsMode",
+                            IBLGLEnvBakerLight.SphericalHarmonicsMode.FAST.name()));
+        }
     }
 
 }

jme3-core/src/main/java/com/jme3/environment/baker/IBLGLEnvBakerLight.java

@@ -44,8 +44,6 @@
 import com.jme3.renderer.RenderManager;
 import com.jme3.scene.Geometry;
 import com.jme3.scene.shape.Box;
-import com.jme3.system.JmeSystem;
-import com.jme3.system.Platform;
 import com.jme3.texture.FrameBuffer;
 import com.jme3.texture.Image;
 import com.jme3.texture.Texture2D;
@@ -72,11 +70,6 @@ public class IBLGLEnvBakerLight extends IBLHybridEnvBakerLight {
      * Selects how spherical harmonics coefficients are baked on the GPU.
      */
     public enum SphericalHarmonicsMode {
-        /**
-         * Use full cubemap integration on desktop and the Hammersley fast path on
-         * Android and iOS.
-         */
-        AUTO,
         /**
          * Use Hammersley sampling.
          */
@@ -87,7 +80,7 @@ public enum SphericalHarmonicsMode {
         QUALITY
     }
 
-    private SphericalHarmonicsMode sphericalHarmonicsMode = SphericalHarmonicsMode.AUTO;
+    private SphericalHarmonicsMode sphericalHarmonicsMode = SphericalHarmonicsMode.FAST;
     private int sphericalHarmonicsFastPathSampleCount = DEFAULT_FAST_SH_SAMPLE_COUNT;
     private Texture2D shCoefTexture;
     private FrameBuffer shBaker;
@@ -210,11 +203,6 @@ public void bakeSphericalHarmonicsCoefficients() {
                 mat.setBoolean("UseFastSphericalHarmonics", false);
                 break;
             }
-            case AUTO: {
-                Platform.Os os = JmeSystem.getPlatform().getOs();
-                mat.setBoolean("UseFastSphericalHarmonics",  os == Platform.Os.Android || os == Platform.Os.iOS);
-                break;
-            }
         }
 
         float remapMaxValue = 0;