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

x3d-tidy decimate #25

Open
coderextreme opened this issue Jan 7, 2025 · 5 comments
Open

x3d-tidy decimate #25

coderextreme opened this issue Jan 7, 2025 · 5 comments
Labels
bug Something isn't working confirmed The issue can be reproduced fixed The issue is already fixed

Comments

@coderextreme
Copy link

Holger, can you look into x3d-tidy conversion of this model? With some cleanup, the model validates in castle-model-viewer and X3DJSAIL https://metagrid1.sv.vt.edu/~bsandbro/gltf/bike/ Sunrize doesn't show anything after converting to .x3d with x3d-tidy.

There were some issues with using | in a DEF and a localDEF EXPORT.

The scene looks so decimated that I'm not surprised nothing shows.

The original glTF doesn't animate in Sunrize (see jet turbine in front). The X3D .x3d file converted by castle-model-converter doesn't animate in Sunrize. Yes, I triedto click the play button.

@create3000
Copy link
Owner

There is one skin object in the file, but no node references this skin, but it should, this result in no humanoid is created. I have made a small fix, that the first skeleton node in the skin, now becomes the humanoid.

@create3000 create3000 added bug Something isn't working fixed The issue is already fixed confirmed The issue can be reproduced labels Jan 7, 2025
@create3000
Copy link
Owner

create3000 commented Jan 7, 2025

It turns out that if there is no skin reference, then no HAnim* nodes should be created, ie. the 'joints' become Transform nodes and can also be animated.

@coderextreme
Copy link
Author

coderextreme commented Jan 7, 2025 via email

@create3000
Copy link
Owner

Good luck, results welcome, next release on Sunday.

@create3000
Copy link
Owner

New version [email protected] is out now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working confirmed The issue can be reproduced fixed The issue is already fixed
Projects
None yet
Development

No branches or pull requests

2 participants