From b4289a2bb7de141fdcb086fa6c9f5b212fc5cd9c Mon Sep 17 00:00:00 2001
From: mistic100 <contact@git.strangeplanet.fr>
Date: Tue, 30 Jul 2024 13:18:43 +0200
Subject: [PATCH] Fix #1394 Closes #1395 performance issues during transitions

- Remove call to Renderer#clearDepth
---
 .../core/src/adapters/EquirectangularAdapter.ts   | 15 ++++++++++++---
 packages/core/src/services/Renderer.ts            |  4 ----
 packages/cubemap-adapter/src/CubemapAdapter.ts    | 15 ++++++++++++---
 .../src/CubemapTilesAdapter.ts                    | 11 ++++++++---
 .../src/EquirectangularTilesAdapter.ts            | 13 +++++++++----
 5 files changed, 41 insertions(+), 17 deletions(-)

diff --git a/packages/core/src/adapters/EquirectangularAdapter.ts b/packages/core/src/adapters/EquirectangularAdapter.ts
index d7904edcf..684dc682c 100644
--- a/packages/core/src/adapters/EquirectangularAdapter.ts
+++ b/packages/core/src/adapters/EquirectangularAdapter.ts
@@ -367,11 +367,20 @@ export class EquirectangularAdapter extends AbstractAdapter<string, Texture, Pan
             -Math.PI / 2
         ).scale(-1, 1, 1);
 
-        return new Mesh(geometry, new MeshBasicMaterial());
+        return new Mesh(geometry);
     }
 
-    setTexture(mesh: EquirectangularMesh, textureData: EquirectangularTexture) {
-        (mesh.material as MeshBasicMaterial).map = textureData.texture;
+    setTexture(mesh: EquirectangularMesh, textureData: EquirectangularTexture, transition?: boolean) {
+        const material = new MeshBasicMaterial();
+
+        material.map = textureData.texture;
+
+        if (transition) {
+            material.depthTest = false;
+            material.depthWrite = false;
+        }
+
+        mesh.material = material;
     }
 
     setTextureOpacity(mesh: EquirectangularMesh, opacity: number) {
diff --git a/packages/core/src/services/Renderer.ts b/packages/core/src/services/Renderer.ts
index ef0df5abd..3f2226126 100644
--- a/packages/core/src/services/Renderer.ts
+++ b/packages/core/src/services/Renderer.ts
@@ -300,10 +300,6 @@ export class Renderer extends AbstractService {
 
         const group = new Group();
         const mesh = this.viewer.adapter.createMesh();
-        mesh.onBeforeRender = function (renderer) {
-            // must be drawn on top
-            renderer.clearDepth();
-        };
         this.viewer.adapter.setTexture(mesh, textureData, true);
         this.viewer.adapter.setTextureOpacity(mesh, 0);
         this.setPanoramaPose(textureData.panoData, mesh);
diff --git a/packages/cubemap-adapter/src/CubemapAdapter.ts b/packages/cubemap-adapter/src/CubemapAdapter.ts
index 7b5a64dcc..ee8133ec3 100644
--- a/packages/cubemap-adapter/src/CubemapAdapter.ts
+++ b/packages/cubemap-adapter/src/CubemapAdapter.ts
@@ -359,10 +359,10 @@ export class CubemapAdapter extends AbstractAdapter<CubemapPanorama, Texture[],
             materials.push(new MeshBasicMaterial());
         }
 
-        return new Mesh(geometry, materials);
+        return new Mesh(geometry, []);
     }
 
-    setTexture(mesh: CubemapMesh, textureData: CubemapTexture) {
+    setTexture(mesh: CubemapMesh, textureData: CubemapTexture, transition?: boolean) {
         const { texture, panoData } = textureData;
 
         for (let i = 0; i < 6; i++) {
@@ -371,7 +371,16 @@ export class CubemapAdapter extends AbstractAdapter<CubemapPanorama, Texture[],
                 texture[i].rotation = Math.PI;
             }
 
-            mesh.material[i].map = texture[i];
+            const material = new MeshBasicMaterial();
+
+            material.map = texture[i];
+
+            if (transition) {
+                material.depthTest = false;
+                material.depthWrite = false;
+            }
+
+            mesh.material.push(material);
         }
     }
 
diff --git a/packages/cubemap-tiles-adapter/src/CubemapTilesAdapter.ts b/packages/cubemap-tiles-adapter/src/CubemapTilesAdapter.ts
index 83f9db4f1..573b9af27 100644
--- a/packages/cubemap-tiles-adapter/src/CubemapTilesAdapter.ts
+++ b/packages/cubemap-tiles-adapter/src/CubemapTilesAdapter.ts
@@ -193,12 +193,12 @@ export class CubemapTilesAdapter extends AbstractAdapter<
     setTexture(mesh: CubemapMesh, textureData: CubemapTexture, transition: boolean) {
         if (transition) {
             this.state.inTransition = true;
-            this.__setTexture(mesh, textureData);
+            this.__setTexture(mesh, textureData, true);
             return;
         }
 
         this.__cleanup();
-        this.__setTexture(mesh, textureData);
+        this.__setTexture(mesh, textureData, false);
 
         this.state.materials = mesh.material;
         this.state.geom = mesh.geometry;
@@ -213,7 +213,7 @@ export class CubemapTilesAdapter extends AbstractAdapter<
         setTimeout(() => this.__refresh());
     }
 
-    private __setTexture(mesh: CubemapMesh, { texture, panoData }: CubemapTexture) {
+    private __setTexture(mesh: CubemapMesh, { texture, panoData }: CubemapTexture, transition: boolean) {
         for (let i = 0; i < 6; i++) {
             let material;
             if (texture) {
@@ -227,6 +227,11 @@ export class CubemapTilesAdapter extends AbstractAdapter<
                 material = new MeshBasicMaterial({ opacity: 0, transparent: true });
             }
 
+            if (transition) {
+                material.depthTest = false;
+                material.depthWrite = false;
+            }
+
             for (let j = 0; j < NB_GROUPS_BY_FACE; j++) {
                 mesh.material.push(material);
             }
diff --git a/packages/equirectangular-tiles-adapter/src/EquirectangularTilesAdapter.ts b/packages/equirectangular-tiles-adapter/src/EquirectangularTilesAdapter.ts
index 0e7d27306..a35d09bcc 100644
--- a/packages/equirectangular-tiles-adapter/src/EquirectangularTilesAdapter.ts
+++ b/packages/equirectangular-tiles-adapter/src/EquirectangularTilesAdapter.ts
@@ -270,17 +270,17 @@ export class EquirectangularTilesAdapter extends AbstractAdapter<
     /**
      * Applies the base texture and starts the loading of tiles
      */
-    setTexture(mesh: EquirectangularMesh, textureData: EquirectangularTexture, transition: boolean) {
+    setTexture(mesh: EquirectangularMesh, textureData: EquirectangularTexture, transition?: boolean) {
         const { texture } = textureData;
 
         if (transition) {
             this.state.inTransition = true;
-            this.__setTexture(mesh, texture);
+            this.__setTexture(mesh, texture, true);
             return;
         }
 
         this.__cleanup();
-        this.__setTexture(mesh, texture);
+        this.__setTexture(mesh, texture, false);
 
         this.state.materials = mesh.material;
         this.state.geom = mesh.geometry;
@@ -295,7 +295,7 @@ export class EquirectangularTilesAdapter extends AbstractAdapter<
         setTimeout(() => this.__refresh());
     }
 
-    private __setTexture(mesh: EquirectangularMesh, texture: Texture) {
+    private __setTexture(mesh: EquirectangularMesh, texture: Texture, transition: boolean) {
         let material;
         if (texture) {
             material = new MeshBasicMaterial({ map: texture });
@@ -303,6 +303,11 @@ export class EquirectangularTilesAdapter extends AbstractAdapter<
             material = new MeshBasicMaterial({ color: this.config.backgroundColor });
         }
 
+        if (transition) {
+            material.depthTest = false;
+            material.depthWrite = false;
+        }
+
         for (let i = 0; i < this.NB_GROUPS; i++) {
             mesh.material.push(material);
         }