chore: remove moduleSettings

visgl · Mar 7, 2024 · 5d73344 · 5d73344
1 parent db20629
commit 5d73344
Show file tree

Hide file tree

Showing 4 changed files with 22 additions and 47 deletions.
diff --git a/examples/tutorials/hello-gltf/app.ts b/examples/tutorials/hello-gltf/app.ts
@@ -66,7 +66,7 @@ export default class AppAnimationLoopTemplate extends AnimationLoopTemplate {
         u_NormalMatrix: new Matrix4(worldMatrix).invert().transpose()
       });
 
-      model.updateModuleSettings({lightSources});
+      // model.updateModuleSettings({lightSources});
       model.draw(renderPass);
     });
     renderPass.end();
@@ -102,7 +102,7 @@ export default class AppAnimationLoopTemplate extends AnimationLoopTemplate {
   }
 }
 
-const lightSources: LightingProps = {
+export const lightSources: LightingProps = {
   ambientLight: {
     color: [255, 133, 133],
     intensity: 1,

diff --git a/examples/tutorials/lighting/app-old.ts b/examples/tutorials/lighting/app-old.ts
@@ -131,22 +131,22 @@ export default class AppAnimationLoopTemplate extends AnimationLoopTemplate {
       fs,
       geometry: new CubeGeometry(),
       modules: [phongMaterial],
-      moduleSettings: {
-        // material: {
-        //   specularColor: [255, 255, 255]
-        // },
-        // lights: [
-        //   {
-        //     type: 'ambient',
-        //     color: [255, 255, 255]
-        //   },
-        //   {
-        //     type: 'point',
-        //     color: [255, 255, 255],
-        //     position: [1, 2, 1]
-        //   }
-        // ]
-      },
+      // moduleSettings: {
+      // material: {
+      //   specularColor: [255, 255, 255]
+      // },
+      // lights: [
+      //   {
+      //     type: 'ambient',
+      //     color: [255, 255, 255]
+      //   },
+      //   {
+      //     type: 'point',
+      //     color: [255, 255, 255],
+      //     position: [1, 2, 1]
+      //   }
+      // ]
+      // },
       bindings: {
         uTexture: texture,
         app: this.uniformStore.getManagedUniformBuffer(device, 'app'),

diff --git a/modules/engine/src/model/model.ts b/modules/engine/src/model/model.ts
@@ -24,7 +24,6 @@ import {getDebugTableForShaderLayout} from '../debug/debug-shader-layout';
 import {debugFramebuffer} from '../debug/debug-framebuffer';
 import {deepEqual} from '../utils/deep-equal';
 import {uid} from '../utils/uid';
-import {splitUniformsAndBindings} from './split-uniforms-and-bindings';
 
 const LOG_DRAW_PRIORITY = 2;
 const LOG_DRAW_TIMEOUT = 10000;
@@ -67,9 +66,6 @@ export type ModelProps = Omit<RenderPipelineProps, 'vs' | 'fs'> & {
 
   transformFeedback?: TransformFeedback;
 
-  /** Mapped uniforms for shadertool modules */
-  moduleSettings?: Record<string, Record<string, any>>;
-
   /** Show shader source in browser? */
   debugShaders?: 'never' | 'errors' | 'warnings' | 'always';
 
@@ -100,7 +96,6 @@ export class Model {
     userData: {},
     defines: {},
     modules: [],
-    moduleSettings: undefined!,
     geometry: null,
     indexBuffer: null,
     attributes: {},
@@ -174,7 +169,6 @@ export class Model {
 
   _attributeInfos: Record<string, AttributeInfo> = {};
   _gpuGeometry: GPUGeometry | null = null;
-  private _getModuleUniforms: (props?: Record<string, Record<string, any>>) => Record<string, any>;
   private props: Required<ModelProps>;
 
   _pipelineNeedsUpdate: string | false = 'newly created';
@@ -213,24 +207,22 @@ export class Model {
     if (isWebGPU && this.props.source) {
       // WGSL
       this.props.shaderLayout ||= getShaderLayoutFromWGSL(this.props.source);
-      const {source, getUniforms} = this.props.shaderAssembler.assembleShader({
+      const {source} = this.props.shaderAssembler.assembleShader({
         platformInfo,
         ...this.props,
         modules
       });
       this.source = source;
-      this._getModuleUniforms = getUniforms;
     } else {
       // GLSL
-      const {vs, fs, getUniforms} = this.props.shaderAssembler.assembleShaderPair({
+      const {vs, fs} = this.props.shaderAssembler.assembleShaderPair({
         platformInfo,
         ...this.props,
         modules
       });
 
       this.vs = vs;
       this.fs = fs;
-      this._getModuleUniforms = getUniforms;
     }
 
     this.vertexCount = this.props.vertexCount;
@@ -290,10 +282,6 @@ export class Model {
     if (props.uniforms) {
       this.setUniforms(props.uniforms);
     }
-    if (props.moduleSettings) {
-      log.warn('Model.props.moduleSettings is deprecated. Use Model.shaderInputs.setProps()')();
-      this.updateModuleSettings(props.moduleSettings);
-    }
     if (props.transformFeedback) {
       this.transformFeedback = props.transformFeedback;
     }
@@ -595,17 +583,6 @@ export class Model {
     this.setNeedsRedraw('uniforms');
   }
 
-  /**
-   * @deprecated Updates shader module settings (which results in uniforms being set)
-   */
-  updateModuleSettings(props: Record<string, any>): void {
-    log.warn('Model.updateModuleSettings is deprecated. Use Model.shaderInputs.setProps()')();
-    const {bindings, uniforms} = splitUniformsAndBindings(this._getModuleUniforms(props));
-    Object.assign(this.bindings, bindings);
-    Object.assign(this.uniforms, uniforms);
-    this.setNeedsRedraw('moduleSettings');
-  }
-
   // Internal methods
 
   /** Get the timestamp of the latest updated bound GPU memory resource (buffer/texture). */

diff --git a/modules/webgl/src/adapter/resources/webgl-render-pipeline.ts b/modules/webgl/src/adapter/resources/webgl-render-pipeline.ts
@@ -12,7 +12,6 @@ import {GL} from '@luma.gl/constants';
 import {getShaderLayout} from '../helpers/get-shader-layout';
 import {withDeviceAndGLParameters} from '../converters/device-parameters';
 import {setUniform} from '../helpers/set-uniform';
-import {splitUniformsAndBindings} from '../../utils/split-uniforms-and-bindings';
 // import {copyUniform, checkUniformValues} from '../../classes/uniforms';
 
 import {WebGLDevice} from '../webgl-device';
@@ -271,11 +270,10 @@ export class WEBGLRenderPipeline extends RenderPipeline {
   // DEPRECATED METHODS
 
   override setUniformsWebGL(uniforms: Record<string, UniformValue>) {
-    const {bindings} = splitUniformsAndBindings(uniforms);
-    Object.keys(bindings).forEach(name => {
+    Object.keys(uniforms).forEach(name => {
       log.warn(
         `Unsupported value "${JSON.stringify(
-          bindings[name]
+          uniforms[name]
         )}" used in setUniforms() for key ${name}. Use setBindings() instead?`
       )();
     });