CesiumGS / obj2gltf

Convert OBJ assets to glTF
Apache License 2.0
1.71k stars 307 forks source link

Converted models are washed out by ambient settings #69

Closed mramato closed 7 years ago

mramato commented 7 years ago

We've discussed this offline, but I couldn't find an issue for it. I've come across quite a few obj files with their ambient values set to 1.0 1.0 1.0, which looks like we translate into additive values in the glTF when they were intended as multiplicative values. The end result is a model with washed out textures.

I believe @lilleyse suggested we keep using additive but discarding "full-white" values of 1.0 1.0 1.0.

I tried to find a clear answer in the obj/mtl spec, but it wasn't very helpful.

These same obj models load fine in Three.js without being washed out, so it's also either ignoring the values or always treating them as multiplicative.

CC @lilleyse @likangning93