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

[new feature] add vertex color morph target #16179

Merged
merged 7 commits into from
Feb 13, 2025

Conversation

kmatzen
Copy link
Contributor

@kmatzen kmatzen commented Feb 12, 2025

Any interest in having a morph target for vertex color? There are some glTF viewers out there that will actually display animated point clouds if you add morph targets for POSITION and COLOR_0. Two that I found are gestaltar and https://gltf-viewer.donmccurdy.com/. Looks like babylon.js doesn't wire in color as a morph target, so I grepped through the code and put it in.

Test plan
I only tried it with a test glb with animated point clouds defined as morph targets. I'm not sure how to correctly test all of this.

@bjsplat
Copy link
Collaborator

bjsplat commented Feb 12, 2025

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

@kmatzen kmatzen changed the title add vertex color morph target [new feature] add vertex color morph target Feb 12, 2025
@bjsplat
Copy link
Collaborator

bjsplat commented Feb 12, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Feb 12, 2025

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

@bjsplat
Copy link
Collaborator

bjsplat commented Feb 12, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Feb 12, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Feb 12, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Feb 12, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Feb 12, 2025

Copy link
Contributor

@deltakosh deltakosh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is very good! thanks a lot for your appreciated contribution

@deltakosh
Copy link
Contributor

cc @alexchuber for the serializer change

@deltakosh
Copy link
Contributor

You have several webgpu tests failing though

Copy link
Contributor

@alexchuber alexchuber left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I reviewed only the glTF exporter-specific file. Just requested a quick change to accomodate RGB vs RGBA color attributes, but other than that, LGTM!

Thanks for the contribution!

@bjsplat
Copy link
Collaborator

bjsplat commented Feb 12, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Feb 12, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Feb 13, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Feb 13, 2025

@kmatzen kmatzen marked this pull request as ready for review February 13, 2025 02:11
@kmatzen kmatzen requested a review from alexchuber February 13, 2025 02:12
@RaananW
Copy link
Member

RaananW commented Feb 13, 2025

@alexchuber , can we merge this?

@alexchuber
Copy link
Contributor

Yes, sorry for the delay. Thanks!

@alexchuber alexchuber merged commit 8453c87 into BabylonJS:master Feb 13, 2025
15 checks passed
@kmatzen kmatzen deleted the kmatzen/morphtargetvertexcolor branch February 13, 2025 19:33
noname0310 added a commit to noname0310/babylon-mmd that referenced this pull request Feb 23, 2025
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

Successfully merging this pull request may close these issues.

6 participants