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

Support for 32 Index format for higher mesh size #1244

Closed
vidurvij-apptronik opened this issue Nov 30, 2023 · 5 comments · Fixed by #1353
Closed

Support for 32 Index format for higher mesh size #1244

vidurvij-apptronik opened this issue Nov 30, 2023 · 5 comments · Fixed by #1353
Labels
enhancement New feature or request

Comments

@vidurvij-apptronik
Copy link

Is your feature request related to a problem? Please describe.
I am trying to import a mesh for my robot in Unity. It has high number of vertices, When I import the mesh directly in MuJoCo, it works but it seems like there is an issue with MuJoCo's unity plugin. It probably stems from the indexformat used to create new meshes in Unity. When I manually edit the StlParser.cs to remove the check for number of vertices and change the format for mesh to IndexFormat.UInt32, its working for MuJoCo version 3.0.1 but not for 2.3.7. I would love to have this supported natively,

Describe the solution you'd like
A clear and concise description of what you want to happen.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Additional context
Add any other context or screenshots about the feature request here.

@vidurvij-apptronik vidurvij-apptronik added the enhancement New feature or request label Nov 30, 2023
@yuvaltassa
Copy link
Collaborator

Would you like to send us a PR?

@vidurvij-apptronik
Copy link
Author

vidurvij-apptronik commented Dec 4, 2023

#1254
@yuvaltassa

@vidurvij-apptronik
Copy link
Author

Any progress on the pull request?

@yuvaltassa
Copy link
Collaborator

We can't accept your pull request without the CLA.
Closing the issue, we can continue the discussion in the PR.

@vidurvij-apptronik
Copy link
Author

vidurvij-apptronik commented Jan 12, 2024

Can I repoen the discussion here! So I need to have a 32 bit index to store the meshes, so we can import higher resolution meshes. I am commenting here because I didn't get any reply there.
@yuvaltassa @Balint-H

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants