-
Notifications
You must be signed in to change notification settings - Fork 727
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
[MJX] Model mesh_vert no longer jax array #1655
Comments
Hi @areiner222 , this got refactored in a large change where You can pin to a prior version of MJ/MJX (likely 3.1.4), would that work for the time being? cc @erikfrey |
Thanks for the update @btaba. Yes will pin a prior version for now - 3.1.3 works just fine and will try 3.1.4. |
I see that you map to the convex mesh inside the collider function and compute face normals, edge directions, unique edges there. Please let me know if I'm not thinking about it correctly! |
Hi @areiner222 , we wouldn't perform the mesh "processing" in jax, as you noted jax puts a very hard limitation on implementing any of those processing functions. We'd precompute and store results in |
I noticed that the mesh vertices (previously geom_convex_verts) of the mjx.Model are no longer jax arrays. I think this precludes me from having different mesh vertices over my vmapped batch dimension which is what I've done historically - use shape parameters to modify a set of convex meshes and recompute inertial properties, etc in jax.
The text was updated successfully, but these errors were encountered: