ShaderMaterial created from MeshStandardMaterial's shader has mirrored reflection?

Questions
1

Hi,

jsfiddle here:
https://jsfiddle.net/qgu17w5o/29/

I’m trying to create a ShaderMaterial based on MeshStandardMaterial (without any tweaks for now, will replace standard.fragmentShader in the future):

    let standard = THREE.ShaderLib['standard'];
    let customMaterial = new THREE.ShaderMaterial({
      lights: true,
      fragmentShader: standard.fragmentShader,
      vertexShader: standard.vertexShader,
      uniforms: THREE.UniformsUtils.clone(standard.uniforms),
    });
    customMaterial.uniforms.diffuse.value = new THREE.Color(0x00ffff);
    customMaterial.uniforms.roughness.value = 0;
    customMaterial.uniforms.metalness.value = 1;
    // ...

It seems working except that the reflections are mirrored.

mirrored

How can I fix this?
Also, am I on the right track to customize MeshStandardMaterial?

Thank you very much!

2

When using ShaderMaterial, common and material specific uniforms are not updated automatically. You have missed to update the uniform flipEnvMap. I’ve added the fix to your code.

https://jsfiddle.net/qgu17w5o/43/

1 Like
3

Thank you!

4

Hi,
I have another question about https://jsfiddle.net/qgu17w5o/43/ :
Why both the following lines are necessry to display the env map?

customMaterial.envMap = hdrCubeRenderTarget.texture;
customMaterial.uniforms.envMap.value = hdrCubeRenderTarget.texture;

Why

customMaterial.uniforms.envMap.value = hdrCubeRenderTarget.texture;

alone is not enough?
Thank you!

5

No. The renderer does perform tests on .envMap so it’s important to set customMaterial.envMap.

6

What about the why?

7

@pailhead the question is:

Why is customMaterial.uniforms.envMap.value = hdrCubeRenderTarget.texture alone not enough?

8

Yes, I’m curious about the why part :slight_smile: The “why” was truncated from mugens reply.

I read this and found it confusing since assigning value to a uniform works on custom shaders.

Maybe this overview can be of some assistance:

My guess is that there is a naming conflict as well as reserved words that perhaps should not be used with ShaderMaterial.