@needle-tools/materialx 1.0.4-next.29f5d7d → 1.0.4-next.8713311

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@needle-tools/materialx",
-  "version": "1.0.4-next.29f5d7d",
+  "version": "1.0.4-next.8713311",
   "type": "module",
   "main": "index.ts",
   "exports": {

package/src/materialx.material.ts

@@ -1,4 +1,4 @@
-import { Camera, DoubleSide, FrontSide, GLSL3, MaterialParameters, Matrix3, Matrix4, Object3D, ShaderMaterial, Texture, Vector3 } from "three";
+import { Camera, DoubleSide, FrontSide, GLSL3, IUniform, MaterialParameters, Matrix3, Matrix4, Object3D, ShaderMaterial, Texture, Vector3 } from "three";
 import { debug } from "./utils.js";
 import { MaterialXEnvironment } from "./materialx.js";
 import { getLightData, getUniformValues, Loaders } from "./materialx.helper.js";
@@ -19,6 +19,8 @@ declare type MaterialXMaterialInitParameters = {
     side?: MaterialParameters['side'],
 }
 
+type Uniforms = Record<string, IUniform & { needsUpdate?: boolean }>;
+
 export class MaterialXMaterial extends ShaderMaterial {
 
     // copy(source: MaterialXMaterial): this {
@@ -146,7 +148,7 @@ export class MaterialXMaterial extends ShaderMaterial {
     envMap: Texture | null = null; // Environment map texture, can be set externally
     updateUniforms = (context: Context, environment: MaterialXEnvironment, object: Object3D, camera: Camera) => {
 
-        const uniforms = this.uniforms;
+        const uniforms = this.uniforms as Uniforms;
 
         // TODO remove. Not sure why this is needed, but without it
         // we currently get some "swimming" where matrices are not up to date.
@@ -155,18 +157,22 @@ export class MaterialXMaterial extends ShaderMaterial {
         // Update standard transformation matrices
         if (uniforms.u_worldMatrix) {
             uniforms.u_worldMatrix.value = object.matrixWorld;
+            uniforms.u_worldMatrix.needsUpdate = true;
         }
 
         if (uniforms.u_viewProjectionMatrix) {
             uniforms.u_viewProjectionMatrix.value.multiplyMatrices(camera.projectionMatrix, camera.matrixWorldInverse);
+            uniforms.u_viewProjectionMatrix.needsUpdate = true;
         }
 
         if (uniforms.u_viewPosition) {
             uniforms.u_viewPosition.value.copy(camera.getWorldPosition(worldViewPos));
+            uniforms.u_viewPosition.needsUpdate = true;
         }
 
         if (uniforms.u_worldInverseTransposeMatrix) {
             uniforms.u_worldInverseTransposeMatrix.value.setFromMatrix3(normalMat.getNormalMatrix(object.matrixWorld));
+            uniforms.u_worldInverseTransposeMatrix.needsUpdate = true;
         }
 
         // Update time uniforms
@@ -185,34 +191,40 @@ export class MaterialXMaterial extends ShaderMaterial {
 
     private updateEnvironmentUniforms = (environment: MaterialXEnvironment) => {
 
+        const uniforms = this.uniforms as Uniforms;
+
         // Get lighting data from environment
         const lightData = environment.lightData || null;
         const lightCount = environment.lightCount || 0;
         const textures = environment.getTextures(this) || null;
 
         // Update light count
-        if (this.uniforms.u_numActiveLightSources && lightCount >= 0) {
-            this.uniforms.u_numActiveLightSources.value = lightCount;
+        if (uniforms.u_numActiveLightSources && lightCount >= 0) {
+            uniforms.u_numActiveLightSources.value = lightCount;
         }
 
         // Update light data
         if (lightData?.length) {
-            this.uniforms.u_lightData.value = lightData;
-            if ("needsUpdate" in this.uniforms.u_lightData && this.uniforms.u_lightData.needsUpdate === false) {
+            uniforms.u_lightData.value = lightData;
+            if ("needsUpdate" in uniforms.u_lightData && uniforms.u_lightData.needsUpdate === false) {
                 if (debug) console.debug(`[MaterialX] LightData assigned (${this.name}, ${this.uuid})`, lightData);
-                this.uniforms.u_lightData.needsUpdate = undefined;
+                uniforms.u_lightData.needsUpdate = undefined;
             }
         }
 
         // Update environment uniforms
-        if (this.uniforms.u_envRadiance) {
-            this.uniforms.u_envRadiance.value = textures.radianceTexture;
+        if (uniforms.u_envRadiance) {
+            const prev = uniforms.u_envRadiance.value;
+            uniforms.u_envRadiance.value = textures.radianceTexture;
+            if (prev != textures.radianceTexture) uniforms.u_envRadiance.needsUpdate = true;
         }
-        if (this.uniforms.u_envRadianceMips) {
-            this.uniforms.u_envRadianceMips.value = Math.trunc(Math.log2(Math.max(textures.radianceTexture?.source.data.width ?? 0, textures.radianceTexture?.source.data.height ?? 0))) + 1;
+        if (uniforms.u_envRadianceMips) {
+            uniforms.u_envRadianceMips.value = Math.trunc(Math.log2(Math.max(textures.radianceTexture?.source.data.width ?? 0, textures.radianceTexture?.source.data.height ?? 0))) + 1;
         }
-        if (this.uniforms.u_envIrradiance) {
-            this.uniforms.u_envIrradiance.value = textures.irradianceTexture;
+        if (uniforms.u_envIrradiance) {
+            const prev = uniforms.u_envIrradiance.value;
+            uniforms.u_envIrradiance.value = textures.irradianceTexture;
+            if (prev != textures.irradianceTexture) uniforms.u_envIrradiance.needsUpdate = true;
         }
 
         this.uniformsNeedUpdate = true;