USDComposer: Fix color space handling for material colors.

mrdoob · mrdoob · commit 73636e35bfc1 · 2026-01-14T13:35:48.000+09:00
diff --git a/examples/jsm/loaders/usd/USDComposer.js b/examples/jsm/loaders/usd/USDComposer.js
@@ -861,7 +861,7 @@ class USDComposer {
 
 				if ( mat.color && mat.color.r === 1 && mat.color.g === 1 && mat.color.b === 1 && ! mat.map ) {
 
-					mat.color.setRGB( displayColor[ 0 ], displayColor[ 1 ], displayColor[ 2 ] );
+					mat.color.setRGB( displayColor[ 0 ], displayColor[ 1 ], displayColor[ 2 ], SRGBColorSpace );
 
 				}
 
@@ -1676,7 +1676,7 @@ class USDComposer {
 
 				if ( Array.isArray( color ) && color.length >= 3 ) {
 
-					material.color.setRGB( color[ 0 ], color[ 1 ], color[ 2 ] );
+					material.color.setRGB( color[ 0 ], color[ 1 ], color[ 2 ], SRGBColorSpace );
 
 				}
 
@@ -1692,7 +1692,7 @@ class USDComposer {
 
 				if ( Array.isArray( color ) && color.length >= 3 ) {
 
-					material.emissive.setRGB( color[ 0 ], color[ 1 ], color[ 2 ] );
+					material.emissive.setRGB( color[ 0 ], color[ 1 ], color[ 2 ], SRGBColorSpace );
 
 				}
 

Original file line number	Diff line number	Diff line change
`@@ -861,7 +861,7 @@ class USDComposer {`
`861`	`861`
`862`	`862`	`if ( mat.color && mat.color.r === 1 && mat.color.g === 1 && mat.color.b === 1 && ! mat.map ) {`
`863`	`863`
`864`		`- mat.color.setRGB( displayColor[ 0 ], displayColor[ 1 ], displayColor[ 2 ] );`
	`864`	`+ mat.color.setRGB( displayColor[ 0 ], displayColor[ 1 ], displayColor[ 2 ], SRGBColorSpace );`
`865`	`865`
`866`	`866`	`}`
`867`	`867`
`@@ -1676,7 +1676,7 @@ class USDComposer {`
`1676`	`1676`
`1677`	`1677`	`if ( Array.isArray( color ) && color.length >= 3 ) {`
`1678`	`1678`
`1679`		`- material.color.setRGB( color[ 0 ], color[ 1 ], color[ 2 ] );`
	`1679`	`+ material.color.setRGB( color[ 0 ], color[ 1 ], color[ 2 ], SRGBColorSpace );`
`1680`	`1680`
`1681`	`1681`	`}`
`1682`	`1682`
`@@ -1692,7 +1692,7 @@ class USDComposer {`
`1692`	`1692`
`1693`	`1693`	`if ( Array.isArray( color ) && color.length >= 3 ) {`
`1694`	`1694`
`1695`		`- material.emissive.setRGB( color[ 0 ], color[ 1 ], color[ 2 ] );`
	`1695`	`+ material.emissive.setRGB( color[ 0 ], color[ 1 ], color[ 2 ], SRGBColorSpace );`
`1696`	`1696`
`1697`	`1697`	`}`
`1698`	`1698`