Skip to content

Commit c970333

Browse files
committed
Merge remote-tracking branch 'aswf/main' into usdshaderball
2 parents 0522440 + b26de54 commit c970333

6 files changed

Lines changed: 28 additions & 19 deletions

File tree

javascript/MaterialXView/package.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,9 @@
1010
"author": "",
1111
"license": "ISC",
1212
"dependencies": {
13+
"fflate": "^0.8.2",
1314
"lil-gui": "^0.19.2",
14-
"three": "^0.152.2"
15+
"three": "^0.183.2"
1516
},
1617
"devDependencies": {
1718
"copy-webpack-plugin": "^14.0.0",

javascript/MaterialXView/source/dropHandling.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import * as THREE from 'three';
2-
import * as fflate from 'three/examples/jsm/libs/fflate.module.js';
2+
import * as fflate from 'fflate';
33

44
const debugFileHandling = false;
55
let loadingCallback = null;

javascript/MaterialXView/source/helper.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -212,10 +212,10 @@ function getWrapping(mode)
212212
*/
213213
function getMinFilter(type, generateMipmaps)
214214
{
215-
const filterType = generateMipmaps ? THREE.LinearMipMapLinearFilter : THREE.LinearFilter;
215+
let filterType = generateMipmaps ? THREE.LinearMipmapLinearFilter : THREE.LinearFilter;
216216
if (type === 0)
217217
{
218-
filterType = generateMipmaps ? THREE.NearestMipMapNearestFilter : THREE.NearestFilter;
218+
filterType = generateMipmaps ? THREE.NearestMipmapNearestFilter : THREE.NearestFilter;
219219
}
220220
return filterType;
221221
}

javascript/MaterialXView/source/index.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -97,6 +97,7 @@ function init()
9797
// Set up renderer
9898
renderer = new THREE.WebGLRenderer({ antialias: true, canvas });
9999
renderer.setSize(window.innerWidth, window.innerHeight);
100+
renderer.outputColorSpace = THREE.SRGBColorSpace;
100101
renderer.debug.checkShaderErrors = false;
101102

102103
window.addEventListener('resize', onWindowResize);

javascript/MaterialXView/source/viewer.js

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,8 @@
44
//
55

66
import * as THREE from 'three';
7-
import { GLTFLoader } from 'three/examples/jsm/loaders/GLTFLoader';
8-
import { RGBELoader } from 'three/examples/jsm/loaders/RGBELoader.js';
7+
import { GLTFLoader } from 'three/examples/jsm/loaders/GLTFLoader.js';
8+
import { HDRLoader } from 'three/examples/jsm/loaders/HDRLoader.js';
99

1010
import { prepareEnvTexture, getLightRotation, findLights, registerLights, getUniformValues } from './helper.js'
1111
import { Group } from 'three';
@@ -36,7 +36,6 @@ export class Scene
3636
{
3737
this._scene = new THREE.Scene();
3838
this._scene.background = new THREE.Color(this.#_backgroundColor);
39-
this._scene.background.convertSRGBToLinear();
4039

4140
const aspectRatio = window.innerWidth / window.innerHeight;
4241
const cameraNearDist = 0.05;
@@ -416,7 +415,6 @@ export class Scene
416415
return this.#_backgroundTexture;
417416
}
418417
var color = new THREE.Color(this.#_backgroundColor);
419-
color.convertSRGBToLinear();
420418
return color;
421419
}
422420

@@ -937,9 +935,10 @@ export class Material
937935

938936
var startUniformUpdate = performance.now();
939937

940-
// Get shaders and uniform values
941-
let vShader = shader.getSourceCode("vertex");
942-
let fShader = shader.getSourceCode("pixel");
938+
// Get shaders and uniform values, removing version directives
939+
// that are already managed by the Three.js runtime.
940+
let vShader = shader.getSourceCode("vertex").replace(/^#version\s+.*\n/, '');
941+
let fShader = shader.getSourceCode("pixel").replace(/^#version\s+.*\n/, '');
943942

944943
let theScene = viewer.getScene();
945944
let flipV = theScene.getFlipGeometryV();
@@ -964,6 +963,7 @@ export class Material
964963
uniforms: uniforms,
965964
vertexShader: vShader,
966965
fragmentShader: fShader,
966+
glslVersion: THREE.GLSL3,
967967
transparent: isTransparent,
968968
blendEquation: THREE.AddEquation,
969969
blendSrc: THREE.OneMinusSrcAlphaFactor,
@@ -1526,7 +1526,7 @@ export class Viewer
15261526
this.materials.push(new Material());
15271527

15281528
this.fileLoader = new THREE.FileLoader();
1529-
this.hdrLoader = new RGBELoader();
1529+
this.hdrLoader = new HDRLoader();
15301530
}
15311531

15321532
//

javascript/package-lock.json

Lines changed: 14 additions & 7 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)