Skip to content

Commit

Permalink
Merge branch 'dev' into transmission-alpha
Browse files Browse the repository at this point in the history
# Conflicts:
#	examples/jsm/loaders/GLTFLoader.js
#	src/renderers/webgl/WebGLProgram.js
  • Loading branch information
mrdoob committed Aug 25, 2021
2 parents 74d7b74 + f058376 commit 831e49e
Show file tree
Hide file tree
Showing 5 changed files with 10 additions and 7 deletions.
3 changes: 1 addition & 2 deletions examples/jsm/loaders/GLTFLoader.js
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@ import {
FileLoader,
FrontSide,
Group,
GLTFOpaqueBlending,
ImageBitmapLoader,
InterleavedBuffer,
InterleavedBufferAttribute,
Expand Down Expand Up @@ -3119,7 +3118,7 @@ class GLTFParser {

} else {

materialParams.blending = GLTFOpaqueBlending;
materialParams.format = RGBFormat;
materialParams.transparent = false;

if ( alphaMode === ALPHA_MODES.MASK ) {
Expand Down
1 change: 1 addition & 0 deletions src/loaders/MaterialLoader.js
Original file line number Diff line number Diff line change
Expand Up @@ -93,6 +93,7 @@ class MaterialLoader extends Loader {
if ( json.side !== undefined ) material.side = json.side;
if ( json.shadowSide !== undefined ) material.shadowSide = json.shadowSide;
if ( json.opacity !== undefined ) material.opacity = json.opacity;
if ( json.format !== undefined ) material.format = json.format;
if ( json.transparent !== undefined ) material.transparent = json.transparent;
if ( json.alphaTest !== undefined ) material.alphaTest = json.alphaTest;
if ( json.depthTest !== undefined ) material.depthTest = json.depthTest;
Expand Down
5 changes: 4 additions & 1 deletion src/materials/Material.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import { EventDispatcher } from '../core/EventDispatcher.js';
import { FrontSide, FlatShading, NormalBlending, LessEqualDepth, AddEquation, OneMinusSrcAlphaFactor, SrcAlphaFactor, AlwaysStencilFunc, KeepStencilOp } from '../constants.js';
import { FrontSide, FlatShading, NormalBlending, LessEqualDepth, AddEquation, OneMinusSrcAlphaFactor, SrcAlphaFactor, AlwaysStencilFunc, KeepStencilOp, RGBAFormat } from '../constants.js';
import * as MathUtils from '../math/MathUtils.js';

let materialId = 0;
Expand All @@ -26,6 +26,7 @@ class Material extends EventDispatcher {
this.vertexColors = false;

this.opacity = 1;
this.format = RGBAFormat;
this.transparent = false;

this.blendSrc = SrcAlphaFactor;
Expand Down Expand Up @@ -301,6 +302,7 @@ class Material extends EventDispatcher {
if ( this.vertexColors ) data.vertexColors = true;

if ( this.opacity < 1 ) data.opacity = this.opacity;
if ( this.format !== RGBAFormat ) data.format = this.format;
if ( this.transparent === true ) data.transparent = this.transparent;

data.depthFunc = this.depthFunc;
Expand Down Expand Up @@ -397,6 +399,7 @@ class Material extends EventDispatcher {
this.vertexColors = source.vertexColors;

this.opacity = source.opacity;
this.format = source.format;
this.transparent = source.transparent;

this.blendSrc = source.blendSrc;
Expand Down
4 changes: 2 additions & 2 deletions src/renderers/webgl/WebGLProgram.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import { WebGLUniforms } from './WebGLUniforms.js';
import { WebGLShader } from './WebGLShader.js';
import { ShaderChunk } from '../shaders/ShaderChunk.js';
import { GLTFOpaqueBlending, NoToneMapping, AddOperation, MixOperation, MultiplyOperation, CubeRefractionMapping, CubeUVRefractionMapping, CubeUVReflectionMapping, CubeReflectionMapping, PCFSoftShadowMap, PCFShadowMap, VSMShadowMap, ACESFilmicToneMapping, CineonToneMapping, CustomToneMapping, ReinhardToneMapping, LinearToneMapping, GammaEncoding, RGBDEncoding, RGBM16Encoding, RGBM7Encoding, RGBEEncoding, sRGBEncoding, LinearEncoding, LogLuvEncoding, GLSL3 } from '../../constants.js';
import { RGBFormat, NoToneMapping, AddOperation, MixOperation, MultiplyOperation, CubeRefractionMapping, CubeUVRefractionMapping, CubeUVReflectionMapping, CubeReflectionMapping, PCFSoftShadowMap, PCFShadowMap, VSMShadowMap, ACESFilmicToneMapping, CineonToneMapping, CustomToneMapping, ReinhardToneMapping, LinearToneMapping, GammaEncoding, RGBDEncoding, RGBM16Encoding, RGBM7Encoding, RGBEEncoding, sRGBEncoding, LinearEncoding, LogLuvEncoding, GLSL3 } from '../../constants.js';

let programIdCount = 0;

Expand Down Expand Up @@ -670,7 +670,7 @@ function WebGLProgram( renderer, cacheKey, parameters, bindingStates ) {
( parameters.toneMapping !== NoToneMapping ) ? getToneMappingFunction( 'toneMapping', parameters.toneMapping ) : '',

parameters.dithering ? '#define DITHERING' : '',
parameters.blending === GLTFOpaqueBlending ? '#define OPAQUE' : '',
parameters.format === RGBFormat ? '#define OPAQUE' : '',

ShaderChunk[ 'encodings_pars_fragment' ], // this code is required here because it is used by the various encoding/decoding function defined below
parameters.map ? getTexelDecodingFunction( 'mapTexelToLinear', parameters.mapEncoding ) : '',
Expand Down
4 changes: 2 additions & 2 deletions src/renderers/webgl/WebGLPrograms.js
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ function WebGLPrograms( renderer, cubemaps, cubeuvmaps, extensions, capabilities
'numDirLights', 'numPointLights', 'numSpotLights', 'numHemiLights', 'numRectAreaLights',
'numDirLightShadows', 'numPointLightShadows', 'numSpotLightShadows',
'shadowMapEnabled', 'shadowMapType', 'toneMapping', 'physicallyCorrectLights',
'doubleSided', 'flipSided', 'numClippingPlanes', 'numClipIntersection', 'depthPacking', 'dithering', 'blending',
'doubleSided', 'flipSided', 'numClippingPlanes', 'numClipIntersection', 'depthPacking', 'dithering', 'format',
'sheenTint', 'transmission', 'transmissionMap', 'thicknessMap'
];

Expand Down Expand Up @@ -257,7 +257,7 @@ function WebGLPrograms( renderer, cubemaps, cubeuvmaps, extensions, capabilities
numClippingPlanes: clipping.numPlanes,
numClipIntersection: clipping.numIntersection,

blending: material.blending,
format: material.format,
dithering: material.dithering,

shadowMapEnabled: renderer.shadowMap.enabled && shadows.length > 0,
Expand Down

0 comments on commit 831e49e

Please sign in to comment.