Skip to content

Box<->Capsule Collision #163

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

Closed
Andrew-Luo1 opened this issue Apr 18, 2025 · 6 comments
Closed

Box<->Capsule Collision #163

Andrew-Luo1 opened this issue Apr 18, 2025 · 6 comments
Assignees

Comments

@Andrew-Luo1
Copy link
Collaborator

Setup

Simulating scene_terrain.xml from #162 using mujoco warp's viewer.

The Issue

Even with nconmax set to 512*1024 in put_data, n_worlds = 1 and shrinking down scene_terrain.xml down to a 3x3 grid of subterrains (versus the 10x20 default), collisions with pyramids don't look correct. (This doesn't happen in cmujoco).

image

image

@camevor
Copy link
Collaborator

camevor commented Apr 22, 2025

Hi @Andrew-Luo1, the capsule-box implementation is still under review (#135). Could you try your scene with that branch?
I just tried it and capsule-box looks good to me, but I ran into some box-box collision errors that I'm looking into right now.

@adenzler-nvidia
Copy link
Collaborator

@Andrew-Luo1 while you're at it - Chris would love to have some feedback on that PR.

@Andrew-Luo1
Copy link
Collaborator Author

Hi @camevor and @adenzler-nvidia something seems to have broken my dev environment between when I PR'd this terrain and the current upstream on the capsule-box branch, i.e. I can't run the viewer. Actually I've recently begun a stint in the LLM world and will have very limited time to work on this project. Apologies and best of luck!

@camevor camevor self-assigned this Apr 24, 2025
@adenzler-nvidia
Copy link
Collaborator

@camevor - can we close this once #208 is merged?

@camevor
Copy link
Collaborator

camevor commented May 5, 2025

Yes, running the apollo scene again with the new box-box and capsule-box, the behavior looks good.

@camevor
Copy link
Collaborator

camevor commented May 13, 2025

The glitches seen no longer occur on main after the merge of Capsule<>Box #135 and the new Box<>Box #208.

@camevor camevor closed this as completed May 13, 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

No branches or pull requests

3 participants