-
Notifications
You must be signed in to change notification settings - Fork 2.5k
Configures mesh collision schemas in convert_mesh.py
#3558
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
Conversation
Signed-off-by: zehao-wang <[email protected]>
Signed-off-by: zehao-wang <[email protected]>
convert_mesh.py
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be good to raise an error if none of the types match (and also avoiding a dangling if-else). Also please run the pre-commit formatter.
Signed-off-by: zehao-wang <[email protected]>
Dear maintainer, this is my first PR—apologies for any missteps. I received notifications about Python dependency and build issues from github notifications, but I only modified two files without touching build tools or those packages. Please let me know if there's anything I need to fix on my end. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM I can close my PR!
convert_mesh.py
convert_mesh.py
# Description The collision approximation configuration changed in main branch, but the code in tools/convert_mesh.py does not sync. Fixes #3557 ## Type of change - Bug fix (non-breaking change which fixes an issue) ## Checklist - [x] I have read and understood the [contribution guidelines](https://isaac-sim.github.io/IsaacLab/main/source/refs/contributing.html) - [x] I have run the [`pre-commit` checks](https://pre-commit.com/) with `./isaaclab.sh --format` - [ ] I have made corresponding changes to the documentation - [x] My changes generate no new warnings - [ ] I have added tests that prove my fix is effective or that my feature works - [ ] I have updated the changelog and the corresponding version in the extension's `config/extension.toml` file - [x] I have added my name to the `CONTRIBUTORS.md` or my name already exists there --------- Signed-off-by: zehao-wang <[email protected]>
Description
The collision approximation configuration changed in main branch, but the code in tools/convert_mesh.py does not sync.
Fixes #3557
Type of change
Checklist
pre-commit
checks with./isaaclab.sh --format
config/extension.toml
fileCONTRIBUTORS.md
or my name already exists there