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

Collision detection doesn't support meshes without index array #9267

Closed
RaananW opened this issue Oct 26, 2020 · 5 comments · Fixed by #9339
Closed

Collision detection doesn't support meshes without index array #9267

RaananW opened this issue Oct 26, 2020 · 5 comments · Fixed by #9339
Assignees
Milestone

Comments

@RaananW
Copy link
Member

RaananW commented Oct 26, 2020

See - https://forum.babylonjs.com/t/collision-problem-bug-with-gltf-mesh-without-indexes/14914

Playground to reproduce:

https://www.babylonjs-playground.com/#SYQW69#505

@sebavan
Copy link
Member

sebavan commented Oct 26, 2020

@Popov72 would you have a bit of bandwith to look into this one ? or should we move it to after 4.2 ?

@Popov72
Copy link
Contributor

Popov72 commented Oct 26, 2020

Trying to finish some things for WebGPU first.

As it's not a bug, I think we are not so much in a hurry(?)

@sebavan
Copy link
Member

sebavan commented Oct 26, 2020

Yup definitely, no rush, let s do before the next bash of webgpu things but after this one :-)

@epajarre
Copy link

epajarre commented Nov 3, 2020

I sort of disagree with the "not a bug". As I see it it is: collision testing works, until it does not.....

I will be doing a workaround to my toolchain for creating the index buffers, this might be a little bit messy as am working with .glb to .glb translation.

With a fast search I did not find a convertToIndexedMesh call in Babylonjs. (please correct me if I just missed it).

@deltakosh deltakosh added this to the 4.2 milestone Nov 3, 2020
@deltakosh
Copy link
Contributor

Yes I confirm the bug status. Please make sure to consider it as soon as possible

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

Successfully merging a pull request may close this issue.

5 participants