pissang / clay-viewer

3D model viewer with high quality rendering and glTF2.0/GLB export
https://pissang.github.io/clay-viewer/editor/
BSD 3-Clause "New" or "Revised" License
771 stars 97 forks source link

Default roughnessFactor value #94

Open Danny-Kint opened 6 years ago

Danny-Kint commented 6 years ago

It seems like, when the roughnessFactor property is not present in the file (for example Assimp doesn't write this value when it's equal to 1.0), ClayViewer interprets this value to 0.0, when it should be 1.0, in this way models are way too metallic. Can you confirm that?

The value for each property (baseColor, metallic, roughness) can be defined using factors or textures. The metallic and roughness properties are packed together in a single texture called metallicRoughnessTexture. If a texture is not given, all respective texture components within this material model are assumed to have a value of 1.0