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

this asset probably refers to invalid image Ids within #129

Closed
pjcozzi opened this issue Dec 24, 2017 · 4 comments
Closed

this asset probably refers to invalid image Ids within #129

pjcozzi opened this issue Dec 24, 2017 · 4 comments
Labels

Comments

@pjcozzi
Copy link
Member

pjcozzi commented Dec 24, 2017

Replaces KhronosGroup/glTF#800 by @xuncanzhe


The error message occurs during the conversion. "Inconsistency error: this asset probably refers to invalid image Ids within ". I can use other software to open the dae file. Could you help me determine the cause of the problem? convert tool or dae file. No exception hint when compiling the [tool.(centos7)

OneDrive file url: https://1drv.ms/u/s!AkXxHoUK23z6hOUF90HpSuQm76K-Vg


Is this invalid COLLADA that the other loaders are just working around?

@xuncanzhe
Copy link

yes, This data is convert from osgb using OpenSceneGraph tool. It can be displayed in Bitmanagement。

@lasalvavida
Copy link
Contributor

I believe this is actually a bug in OpenCOLLADA. The image is declared inside of the profile_COMMON tag instead of library_images, but that should still be valid:

image

The new converter doesn't crash on this, just exports it without the texture. If you put it where OpenCOLLADA expects, it comes out properly:

image

OpenCOLLADA should call writeImage() on our writer for the texture image, but that never happens.

@lasalvavida
Copy link
Contributor

Opened an issue in OpenCOLLADA: KhronosGroup/OpenCOLLADA#571

This can be closed once this gets fixed on their end and that gets merged in here.

@lasalvavida
Copy link
Contributor

I have an open pull request in OpenCOLLADA for this issue: KhronosGroup/OpenCOLLADA#578

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

No branches or pull requests

3 participants