From dbd67dea83c71d1d7d44df017bf9509e74788ab2 Mon Sep 17 00:00:00 2001 From: Takahiro Date: Tue, 7 Mar 2017 09:05:55 -0800 Subject: [PATCH] Add the comment about glTF metallic texture channel into GLTF2Loader --- examples/js/loaders/GLTF2Loader.js | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/examples/js/loaders/GLTF2Loader.js b/examples/js/loaders/GLTF2Loader.js index 276cd1efef..63c96d0208 100644 --- a/examples/js/loaders/GLTF2Loader.js +++ b/examples/js/loaders/GLTF2Loader.js @@ -1230,6 +1230,13 @@ THREE.GLTF2Loader = ( function () { var textureIndex = metallicRoughness.metallicRoughnessTexture.index; + // Note that currently metalnessMap would be entirely ignored because + // Three.js and glTF specification use different texture channels for metalness + // (Blue: Three.js, Red: glTF). + // But glTF specification team is discussing if they can change. + // Let's keep an eye on it so far. + // + // https://github.com/KhronosGroup/glTF/issues/857 materialParams.metalnessMap = dependencies.textures[ textureIndex ]; materialParams.roughnessMap = dependencies.textures[ textureIndex ]; -- GitLab