Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Textures are not fully matched #234

Open
CrashedBboy opened this issue Jan 16, 2019 · 3 comments
Open

Textures are not fully matched #234

CrashedBboy opened this issue Jan 16, 2019 · 3 comments

Comments

@CrashedBboy
Copy link

CrashedBboy commented Jan 16, 2019

Hello, I'm trying to convert this collada model into GLB.

I used the command below to convert it (in Ubuntu):

COLLADA2GLTF-bin -i 4833_9.dae -o 4833_9.glb -b

They were converted without any error, but when I load the output GLB into Cesium.js (version 1.53),
It seems that some area of the model are not matched to the textures and shows in black.

in cesium

In order to make sure the original collada file is valid, I've also load the model in Google Earth, and it looks normal.

in google earth

I'm not sure which part in the whole converting process goes wrong.

Thanks in advance!

@lasalvavida
Copy link
Contributor

Re-hosting the relevant model here so that it is directly available:
collada-model-20190116T161419Z-001.zip

@lasalvavida
Copy link
Contributor

lasalvavida commented Jan 16, 2019

This may be an issue with your COLLADA model.

Some of your textures are not sized to be powers of two. This isn't critical, but I would consider resizing those textures.

It looks like the output glTF has some invalid normal vectors which would explain the inconsistent lighting in the renders I have included below. I will dig into this more when I have time to make sure that it isn't a conversion issue, but it is possible that something is incorrect in the normal data from the source COLLADA.

Validator

Error Message Pointer
ACCESSOR_NON_UNIT 4 accessor elements not of unit length: 0. [AGGREGATED] /accessors/1
ACCESSOR_NON_UNIT 32 accessor elements not of unit length: 0. [AGGREGATED] /accessors/5
ACCESSOR_NON_UNIT 31 accessor elements not of unit length: 0. [AGGREGATED] /accessors/9
Info Message Pointer
IMAGE_NPOT_DIMENSIONS Image has non-power-of-two dimensions: 85x194. /images/0
IMAGE_NPOT_DIMENSIONS Image has non-power-of-two dimensions: 106x64. /images/1
IMAGE_NPOT_DIMENSIONS Image has non-power-of-two dimensions: 132x64. /images/2
ACCESSOR_INDEX_TRIANGLE_DEGENERATE Indices accessor contains 1 degenerate triangles. /accessors/0

Babylon

convert-babylon

Cesium

convert-cesium

Three

convert-three

@CrashedBboy
Copy link
Author

Thanks for your advice, I will check the normal vector data in my collada files.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants