djsundog/obj2gltf
1
0
Fork 0
mirror of https://github.com/CesiumGS/obj2gltf.git synced 2025-01-31 03:53:13 -05:00
Code Issues Packages Projects Releases Wiki Activity

Fix ambient of 1,1,1

Browse Source
This commit is contained in:
Sean Lilley 2017-04-23 13:34:09 -04:00
parent 9afaf48a61
commit 20bbd17cd9
2 changed files with 16 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
lib/createGltf.js
View File

@ -86,6 +86,13 @@ function createGltf(objData) {
transparent = diffuse[3] < 1.0;
}
if (Array.isArray(ambient)) {
// If ambient color is [1, 1, 1] assume it is a multiplier and instead change to [0, 0, 0]
if (ambient[0] === 1.0 && ambient[1] === 1.0 && ambient[2] === 1.0) {
ambient = [0.0, 0.0, 0.0, 1.0];
}
}
var doubleSided = transparent;
if (!hasNormals) {

9
specs/lib/createGltfSpec.js
View File

@ -359,4 +359,13 @@ describe('createGltf', function() {
var positionAccessor = gltf.accessors[primitive.attributes.POSITION];
expect(positionAccessor.count).toBe(vertexCount);
});
it('ambient of [1, 1, 1] is treated as [0, 0, 0]', function() {
boxObjData.materials.Material.ambientColor = [1.0, 1.0, 1.0, 1.0];
var gltf = createGltf(boxObjData);
var ambient = gltf.materials.Material.extensions.KHR_materials_common.values.ambient;
expect(ambient).toEqual([0.0, 0.0, 0.0, 1.0]);
});
});