ConvexPolyhedron() has empty vertices and faces arrays when created, and has a vsub error

Once passing a Geometry with verticies and faces to ConvexPolyhedron, the arrays of both get stripped, this breaks the ability to use the ConvexPolyhedron. Another issue is that ConvexGeometry strips the bounding shapes (both box and sphere), not sure if this is causing the error TypeError: Cannot read property 'vsub' of undefined at Function.ConvexPolyhedron.computeNormal

An example is here https://codesandbox.io/s/convexpolyhedron-test-pjwrr?runonclick=1&file=/src/index.js

To see the issue, toggle the bool on line 58

[kyleatblackfoot]

@codynova @stockHuman

I did check use-cannon to see why ConvexPolyhedron worked for an imported GLTF model, where cannon-es fails. I understand that ConvexPolyhedron doesn’t take THREE.Geometry or its raw data.

It’s dawning on me though that shaping the face array can be done before creating the new ConvexPolyherdron and doesn’t have to be done within cannon-es script itself. The reason I initially overlooked this is because use-cannon does this check, whereas cannon-es does not.

So to confirm, ConvexPolyhedron is expecting Vec3 for vertices and and all Face3s converted a multidimensional array before its even passed to ConvexPolyhedron.

I do propose in a feature request in that case, that perhaps in those four methods, that there is a conditional check to make it easier for the end user to use a THREE.Geometry directly by doing these checks inside the script base, how worthwhile that is, perhaps not very much so. Worth the thought though.

[arpu]

ok looks like this is working with BufferGeometry

@kyleatblackfoot https://codesandbox.io/s/trimesh-test-ixvb0