From dca5c31ad0581261e48c6b49bf3022ad3210c216 Mon Sep 17 00:00:00 2001
From: alvarosabu <alvaro.saburido@gmail.com>
Date: Sun, 17 Dec 2023 16:24:25 +0100
Subject: [PATCH] fix: reduce lagging on the reflection

---
 src/core/controls/OrbitControls.vue                 |  2 +-
 src/core/materials/meshReflectionMaterial/index.vue | 13 +++++++------
 2 files changed, 8 insertions(+), 7 deletions(-)

diff --git a/src/core/controls/OrbitControls.vue b/src/core/controls/OrbitControls.vue
index 376d59ae..08763792 100644
--- a/src/core/controls/OrbitControls.vue
+++ b/src/core/controls/OrbitControls.vue
@@ -234,7 +234,7 @@ const props = withDefaults(defineProps<OrbitControlsProps>(), {
   makeDefault: false,
   autoRotate: false,
   autoRotateSpeed: 2,
-  enableDamping: false,
+  enableDamping: true,
   dampingFactor: 0.05,
   enablePan: true,
   keyPanSpeed: 7,
diff --git a/src/core/materials/meshReflectionMaterial/index.vue b/src/core/materials/meshReflectionMaterial/index.vue
index 028ca516..a80ef03d 100644
--- a/src/core/materials/meshReflectionMaterial/index.vue
+++ b/src/core/materials/meshReflectionMaterial/index.vue
@@ -1,5 +1,5 @@
 <script setup lang="ts">
-import { shallowRef, onBeforeUnmount, watchEffect, toRefs, reactive, computed, useAttrs } from 'vue'
+import { shallowRef, onBeforeUnmount, watchEffect, toRefs, shallowReactive, computed, useAttrs } from 'vue'
 import { useRenderLoop, useTresContext } from '@tresjs/core'
 import type { TresObject } from '@tresjs/core'
 import type {
@@ -171,7 +171,7 @@ const computedBlur = computed(() => {
 
 const hasBlur = computed(() => computedBlur.value[0] > 0 || computedBlur.value[1] > 0)
 
-const state = reactive({
+const state = shallowReactive({
   reflectorPlane: new Plane(),
   normal: new Vector3(),
   reflectorWorldPosition: new Vector3(),
@@ -188,8 +188,6 @@ const state = reactive({
 
 watchEffect(() => {
   fbo.value?.texture.dispose()
-  fbo.value?.dispose()
-  outputFbo.value?.dispose()
   
   const parameters = {
     minFilter: LinearFilter,
@@ -255,11 +253,13 @@ onLoop(() => {
   const parent: TresObject = findMeshByMaterialUuid(scene.value as unknown as TresObject, materialRef.value.uuid)
   if (!parent) return
   
-  beforeRender(parent)
-
   parent.visible = false // Avoid re-rendering the reflected object
   const currentXrEnabled = renderer.value.xr.enabled
   const currentShadowAutoUpdate = renderer.value.shadowMap.autoUpdate
+  
+  console.time('beforeRender')
+  beforeRender(parent)
+  console.timeEnd('beforeRender')
 
   renderer.value.shadowMap.autoUpdate = false
   renderer.value.setRenderTarget(fbo.value)
@@ -272,6 +272,7 @@ onLoop(() => {
   renderer.value.shadowMap.autoUpdate = currentShadowAutoUpdate
   parent.visible = true
   renderer.value.setRenderTarget(null)
+
 })
 
 const attrs = useAttrs()