- 28 Aug, 2025 3 commits
-
-
ooctipus authored
# Description The test priniting, sometimes gets super verbose and non-readable when isaacsim or kit are in bad state. since we only care about failure test cases and those printing, we can disable --verbose flag ## Type of change <!-- As you go through the list, delete the ones that are not applicable. --> - Bug fix (non-breaking change which fixes an issue) - New feature (non-breaking change which adds functionality) - Breaking change (fix or feature that would cause existing functionality to not work as expected) - This change requires a documentation update ## Screenshots Please attach before and after screenshots of the change if applicable. <!-- Example: | Before | After | | ------ | ----- | | _gif/png before_ | _gif/png after_ | To upload images to a PR -- simply drag and drop an image while in edit mode and it should upload the image directly. You can then paste that source into the above before/after sections. --> ## Checklist - [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 <!-- As you go through the checklist above, you can mark something as done by putting an x character in it For example, - [x] I have done this task - [ ] I have not done this task --> --------- Co-authored-by:
Kelly Guo <kellyguo123@hotmail.com> -
Kelly Guo authored
# Description We have been hunting down a strange issue in distributed training setups with rendering enabled, where often the process would hang midway through training and causes NCCL timeouts. A workaround was discovered to set `app.execution.debug.forceSerial = true`, which forces serialized scheduling of omni graph within the same thread. This appears to have resolved the hanging issue and did not cause performance regressions. ## Type of change <!-- As you go through the list, delete the ones that are not applicable. --> - Bug fix (non-breaking change which fixes an issue) ## Checklist - [x] I have run the [`pre-commit` checks](https://pre-commit.com/) with `./isaaclab.sh --format` - [x] 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 - [ ] I have added my name to the `CONTRIBUTORS.md` or my name already exists there <!-- As you go through the checklist above, you can mark something as done by putting an x character in it For example, - [x] I have done this task - [ ] I have not done this task -->
-
Francisco Beltrao authored
# Description Corrected the file path for the list_envs.py script in the documentation. Fixes # (issue) ## Type of change - Fix a typo ## Checklist - [x] I have run the [`pre-commit` checks](https://pre-commit.com/) with `./isaaclab.sh --format` - [x] I have made corresponding changes to the documentation - [x] My changes generate no new warnings - [x] I have added tests that prove my fix is effective or that my feature works - [x] I have updated the changelog and the corresponding version in the extension's `config/extension.toml` file - [ ] I have added my name to the `CONTRIBUTORS.md` or my name already exists there Signed-off-by:
Francisco Beltrao <fbeltrao@windowslive.com>
-
- 27 Aug, 2025 3 commits
-
-
michaellin6 authored
# Description <!-- Thank you for your interest in sending a pull request. Please make sure to check the contribution guidelines. Link: https://isaac-sim.github.io/IsaacLab/main/source/refs/contributing.html --> Enhance Pink IK Controller with Null Space Posture Control This PR improves the Pink IK controller integration for better humanoid robot control and more natural postures. **Note**: Original this PR was staged in the internal repo (#547). It has been moved here due to new Github workflow. ## Key Changes ### New Null Space Posture Task - Added NullSpacePostureTask to enforce postural constraints on shoulder/waist joints while prioritizing end-effector tasks - Maintains natural robot poses during manipulation ### Controller Improvements - Tuned low level PD controller gains - Support mixed task types (FrameTask + NullSpacePostureTask) ### Testing & Environment Updates - Redesigned pink controller test script to use JSON-based configurations to program test motions. - Updated all environments (PickPlace, NutPour, ExhaustPipe) with null space control, damping tasks, and improved tracking - Added `Isaac-PickPlace-GR1T2-WaistEnabled-Abs-v0` env that is identical to `Isaac-PickPlace-GR1T2-Abs-v0` but enables the Waist DOFs. - Added target_eef_link_names mapping for clearer link specification Fixes # (issue) These changes help fix the following problems from [VDR feedback](https://docs.google.com/document/d/1saB1QA5r_WlD1l17q7C04WWNltnBW-K0ydI2UB8jxAs/edit?tab=t.0) - [Enable Waist DOF](https://nvbugspro.nvidia.com/bug/5235527) - Discourage elbow flare - Make controller low-latency and low-jerk. **We improved the unit test for the pink controller and reduced our position and rotation accuracy tolerance from 30 mm, 10 degrees to 1 mm, 1 degree.** - Develop metric for controller performance - Added a flag to disable failure due to joint limits. Previously, any commanded pose that ended in joint limit violation would result in no solution and the controlled robot freezing in place. This change gets the solver to still provide a solution and instead issue a warning for joint limit violations. ## Screenshots These controller changes have been tested through the Mimic pipeline (teleop_se3_agent.py, record/replay_demos.py). Here are videos showing teleoperation of all three environments working. ### PickPlace-GR1T2-Abs  ### NutPour-GR1T2  ### ExhaustPipe-GR1T2  ### Successfully Trained Robomimic Model Rollout on PickPlace task For the two robomimic tasks: `Isaac-PickPlace-GR1T2-Abs-v0` and `Isaac-NutPour-GR1T2-Pink-IK-Abs-v0`, if we collect a new dataset, we achieve a success rate of 96 and 92% respectively.   ## Checklist - [x] I have run the [`pre-commit` checks](https://pre-commit.com/) with `./isaaclab.sh --format` - [x] I have made corresponding changes to the documentation - [x] My changes generate no new warnings - [x] I have added tests that prove my fix is effective or that my feature works - [x] 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 <!-- As you go through the checklist above, you can mark something as done by putting an x character in it For example, - [x] I have done this task - [ ] I have not done this task --> --------- Co-authored-by:
Kelly Guo <kellyg@nvidia.com> -
ooctipus authored
This PR fixes typo in isaaclab.bat Should be `!allArgs!` instead of `!allArgs1` ## Checklist - [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 <!-- As you go through the checklist above, you can mark something as done by putting an x character in it For example, - [x] I have done this task - [ ] I have not done this task -->
-
ooctipus authored
# Description If Isaac Lab is installed through pip, tasks generated with the Internal template cannot be placed inside the site-packages directory of that installation. Since this setup would never work correctly, it makes no sense to offer the Internal template as an option. This PR therefore disables that option entirely whenever Isaac Lab is detected as running from a pip-installed path. Fixes # (issue) ## Type of change <!-- As you go through the list, delete the ones that are not applicable. --> - Bug fix (non-breaking change which fixes an issue) - This change requires a documentation update <!-- Example: | Before | After | | ------ | ----- | | _gif/png before_ | _gif/png after_ | To upload images to a PR -- simply drag and drop an image while in edit mode and it should upload the image directly. You can then paste that source into the above before/after sections. --> ## Checklist - [x] I have run the [`pre-commit` checks](https://pre-commit.com/) with `./isaaclab.sh --format` - [x] 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 <!-- As you go through the checklist above, you can mark something as done by putting an x character in it For example, - [x] I have done this task - [ ] I have not done this task -->
-
- 26 Aug, 2025 2 commits
-
-
Kelly Guo authored
# Description Fixes a broken link in the docs to point to the feature/newton branch. ## Type of change <!-- As you go through the list, delete the ones that are not applicable. --> - Bug fix (non-breaking change which fixes an issue) - This change requires a documentation update ## Checklist - [x] I have run the [`pre-commit` checks](https://pre-commit.com/) with `./isaaclab.sh --format` - [x] 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 - [ ] I have added my name to the `CONTRIBUTORS.md` or my name already exists there <!-- As you go through the checklist above, you can mark something as done by putting an x character in it For example, - [x] I have done this task - [ ] I have not done this task -->
-
Michael Gussert authored
Adds a new section in the docs for experimental features. As the first experimental feature of Isaac Lab, we are also including instructions for running a feature branch of Isaac lab with Newton. This change adds in the initial set of documentation for early Newton support for Isaac lab. ## Checklist - [ ] I have run the [`pre-commit` checks](https://pre-commit.com/) with `./isaaclab.sh --format` - [ ] I have made corresponding changes to the documentation - [ ] 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 - [ ] I have added my name to the `CONTRIBUTORS.md` or my name already exists there --------- Co-authored-by:
Gavriel State <gavrielstate@gmail.com>
Co-authored-by:
Milad-Rakhsha <miladrakhsha@gmail.com>
Co-authored-by:
Milad Rakhsha <mrakhsha@nvidia.com>
Co-authored-by:
Kelly Guo <kellyg@nvidia.com>
-
- 25 Aug, 2025 1 commit
-
-
rwiltz authored
# Description Performance overhead varies based on the profiler backend even when profiling is not enabled. In limited testing with AR profile enabled, this yields about 5% improvement in framerate. Follow up of #3238 <!-- Thank you for your interest in sending a pull request. Please make sure to check the contribution guidelines. Link: https://isaac-sim.github.io/IsaacLab/main/source/refs/contributing.html --> <!-- As a practice, it is recommended to open an issue to have discussions on the proposed pull request. This makes it easier for the community to keep track of what is being developed or added, and if a given feature is demanded by more than one party. --> ## Type of change <!-- As you go through the list, delete the ones that are not applicable. --> - Bug fix (non-breaking change which fixes an issue) - New feature (non-breaking change which adds functionality) - Breaking change (fix or feature that would cause existing functionality to not work as expected) - This change requires a documentation update ## Screenshots Please attach before and after screenshots of the change if applicable. <!-- Example: | Before | After | | ------ | ----- | | _gif/png before_ | _gif/png after_ | To upload images to a PR -- simply drag and drop an image while in edit mode and it should upload the image directly. You can then paste that source into the above before/after sections. --> ## Checklist - [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 - [ ] I have added my name to the `CONTRIBUTORS.md` or my name already exists there <!-- As you go through the checklist above, you can mark something as done by putting an x character in it For example, - [x] I have done this task - [ ] I have not done this task -->
-
- 23 Aug, 2025 1 commit
-
-
ben-johnston-nv authored
# Description This change prepares the unit test framework to allow running CI within the Isaac Sim pipeline to catch issues earlier on in Isaac Sim changes. <!-- As a practice, it is recommended to open an issue to have discussions on the proposed pull request. This makes it easier for the community to keep track of what is being developed or added, and if a given feature is demanded by more than one party. --> ## Type of change <!-- As you go through the list, delete the ones that are not applicable. --> - New feature (non-breaking change which adds functionality) ## Checklist - [x] I have run the [`pre-commit` checks](https://pre-commit.com/) with `./isaaclab.sh --format` - [x] I have made corresponding changes to the documentation - [x] My changes generate no new warnings - [x] I have added tests that prove my fix is effective or that my feature works - [x] 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 <!-- As you go through the checklist above, you can mark something as done by putting an x character in it For example, - [x] I have done this task - [ ] I have not done this task -->
-
- 22 Aug, 2025 12 commits
-
-
lotusl-code authored
# Description In document, add a chart to direct the user which version to pull. ## Type of change - This change requires a documentation update ## Checklist - [ ] I have run the [`pre-commit` checks](https://pre-commit.com/) with `./isaaclab.sh --format` - [ ] I have made corresponding changes to the documentation - [ ] 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 - [ ] I have added my name to the `CONTRIBUTORS.md` or my name already exists there
-
matthewtrepte authored
# Description <!-- Thank you for your interest in sending a pull request. Please make sure to check the contribution guidelines. Link: https://isaac-sim.github.io/IsaacLab/main/source/refs/contributing.html --> Add periodic logs when checking if a USD path exists on a Nucleus Server. Improves user experience when the check takes a while, usually because of no internet connection or the file doesn't exist. <!-- As a practice, it is recommended to open an issue to have discussions on the proposed pull request. This makes it easier for the community to keep track of what is being developed or added, and if a given feature is demanded by more than one party. --> ## Type of change <!-- As you go through the list, delete the ones that are not applicable. --> - New feature (non-breaking change which adds functionality) ## Screenshots Please attach before and after screenshots of the change if applicable. <!-- Example: | Before | After | | ------ | ----- | | _gif/png before_ | _gif/png after_ | To upload images to a PR -- simply drag and drop an image while in edit mode and it should upload the image directly. You can then paste that source into the above before/after sections. --> ## Checklist - [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 - [x] I have updated the changelog and the corresponding version in the extension's `config/extension.toml` file - [ ] I have added my name to the `CONTRIBUTORS.md` or my name already exists there <!-- As you go through the checklist above, you can mark something as done by putting an x character in it For example, - [x] I have done this task - [ ] I have not done this task --> --------- Signed-off-by:
matthewtrepte <mtrepte@nvidia.com>
Signed-off-by:
Kelly Guo <kellyg@nvidia.com>
Co-authored-by:
Kelly Guo <kellyg@nvidia.com> -
Xinpeng Liu authored
# Description <!-- Thank you for your interest in sending a pull request. Please make sure to check the contribution guidelines. Link: https://isaac-sim.github.io/IsaacLab/main/source/refs/contributing.html --> In function `isaaclab.envs.mdp.events.randomize_rigid_body_com`, the random CoM values were sampled with respect to `env_ids`, but when added to the bodies' current CoMs, `env_ids` is omitted, resulting in potential shape mismatch error if `len(env_ids) <> env.scene.num_envs`. <!-- As a practice, it is recommended to open an issue to have discussions on the proposed pull request. This makes it easier for the community to keep track of what is being developed or added, and if a given feature is demanded by more than one party. --> ## Type of change <!-- As you go through the list, delete the ones that are not applicable. --> - Bug fix (non-breaking change which fixes an issue) ## Checklist - [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:
Xinpeng Liu <43176336+Foruck@users.noreply.github.com>
Co-authored-by:
ooctipus <zhengyuz@nvidia.com> -
Ossama Ahmed authored
# Description Fixed setting friction coeffecients in the articulation classs Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change. Fixes #3231 Fixed setting friction coeffecients in the articulation classs ## Type of change - Bug fix (non-breaking change which fixes an issue) ## Checklist - [X] I have run the [`pre-commit` checks](https://pre-commit.com/) with `./isaaclab.sh --format` - [X] I have made corresponding changes to the documentation - [X] My changes generate no new warnings - [X] I have added tests that prove my fix is effective or that my feature works - [X] 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:
ooctipus <zhengyuz@nvidia.com>
Co-authored-by:
ooctipus <zhengyuz@nvidia.com> -
Louis LE LAY authored
# Description Replace the incorrect env ID `Isaac-Stack-Cube-Franka-Blueprint-v0` with the correct `Isaac-Stack-Cube-Franka-IK-Rel-Blueprint-v0` on the environments list page to match what’s registered in code and shown by `scripts/environments/list_envs.py`. Fixes https://github.com/isaac-sim/IsaacLab/issues/3207 ## Type of change <!-- As you go through the list, delete the ones that are not applicable. --> - Bug fix (non-breaking change which fixes an issue) - This change requires a documentation update ## Checklist - [x] I have run the [`pre-commit` checks](https://pre-commit.com/) with `./isaaclab.sh --format` - [x] 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 Co-authored-by:
louislelay <louislelay@pal-robotics.com>
Co-authored-by:
ooctipus <zhengyuz@nvidia.com> -
Ziqi Fan authored
# Description Fix MuJoCo link <img width="1726" height="1220" alt="image" src="https://github.com/user-attachments/assets/fed03af9-cc0c-4b94-8a8c-42ecd5992f6c" /> ## Type of change - This change requires a documentation update ## Checklist - [ ] I have run the [`pre-commit` checks](https://pre-commit.com/) with `./isaaclab.sh --format` - [x] I have made corresponding changes to the documentation - [ ] 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 Co-authored-by:
James Tigue <166445701+jtigue-bdai@users.noreply.github.com> -
ooctipus authored
# Description This PR adds stricter interpretation rules to value specified in sb3_ppo_cfg.yaml, disallowing eval on any dict, which my contain arbitrary code that makes program vulnerable. Now, `eval` is got rid of, only str that start with `nn.` can be used to only import the module from torch.nn. That seems to cover all usage for lab so far, I can make more accommodations if there are more cases but it seems like it is currently sufficient. Fixes # (issue) <!-- As a practice, it is recommended to open an issue to have discussions on the proposed pull request. This makes it easier for the community to keep track of what is being developed or added, and if a given feature is demanded by more than one party. --> ## Type of change <!-- As you go through the list, delete the ones that are not applicable. --> - Bug fix (non-breaking change which fixes an issue) ## Screenshots Please attach before and after screenshots of the change if applicable. <!-- Example: | Before | After | | ------ | ----- | | _gif/png before_ | _gif/png after_ | To upload images to a PR -- simply drag and drop an image while in edit mode and it should upload the image directly. You can then paste that source into the above before/after sections. --> ## Checklist - [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 - [x] 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 <!-- As you go through the checklist above, you can mark something as done by putting an x character in it For example, - [x] I have done this task - [ ] I have not done this task --> --------- Signed-off-by:
ooctipus <zhengyuz@nvidia.com>
Co-authored-by:
Mayank Mittal <12863862+Mayankm96@users.noreply.github.com>
Co-authored-by:
Kelly Guo <kellyg@nvidia.com> -
Sixiang Chen authored
# Description This PR fixes a bug in the observation space construction for non-concatenated groups in `ManagerBasedRLEnv._configure_gym_env_spaces` method. Previously, only the last term in each group was included in the Dict, causing loss of observation information. Now, all terms are correctly added to the group Dict. Fixes #3133 <!-- As a practice, it is recommended to open an issue to have discussions on the proposed pull request. This makes it easier for the community to keep track of what is being developed or added, and if a given feature is demanded by more than one party. --> ## Type of change <!-- As you go through the list, delete the ones that are not applicable. --> - Bug fix (non-breaking change which fixes an issue) <!-- Example: | Before | After | | ------ | ----- | | _gif/png before_ | _gif/png after_ | To upload images to a PR -- simply drag and drop an image while in edit mode and it should upload the image directly. You can then paste that source into the above before/after sections. -->
-
James Tigue authored
# Description Adds tests for SensorBase ## Type of change <!-- As you go through the list, delete the ones that are not applicable. --> - New feature (non-breaking change which adds functionality) ## Checklist - [ ] I have run the [`pre-commit` checks](https://pre-commit.com/) with `./isaaclab.sh --format` - [ ] I have made corresponding changes to the documentation - [ ] 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 - [ ] I have added my name to the `CONTRIBUTORS.md` or my name already exists there <!-- As you go through the checklist above, you can mark something as done by putting an x character in it For example, - [x] I have done this task - [ ] I have not done this task --> --------- Signed-off-by:
James Tigue <166445701+jtigue-bdai@users.noreply.github.com>
Signed-off-by:
ooctipus <zhengyuz@nvidia.com>
Co-authored-by:
Kelly Guo <kellyg@nvidia.com>
Co-authored-by:
ooctipus <zhengyuz@nvidia.com> -
matthewtrepte authored
Update render preset balanced and performance modes to rtx.indirrectDiffuse.enabled to false (#3240) # Description <!-- Thank you for your interest in sending a pull request. Please make sure to check the contribution guidelines. Link: https://isaac-sim.github.io/IsaacLab/main/source/refs/contributing.html --> Update render preset balanced and performance modes to rtx.indirrectDiffuse.enabled to false, the original value from Isaac Lab 2.1. <!-- As a practice, it is recommended to open an issue to have discussions on the proposed pull request. This makes it easier for the community to keep track of what is being developed or added, and if a given feature is demanded by more than one party. --> ## Type of change <!-- As you go through the list, delete the ones that are not applicable. --> - Bug fix (non-breaking change which fixes an issue) - New feature (non-breaking change which adds functionality) - Breaking change (fix or feature that would cause existing functionality to not work as expected) - This change requires a documentation update ## Screenshots Please attach before and after screenshots of the change if applicable. <!-- Example: | Before | After | | ------ | ----- | | _gif/png before_ | _gif/png after_ | To upload images to a PR -- simply drag and drop an image while in edit mode and it should upload the image directly. You can then paste that source into the above before/after sections. --> ## Checklist - [ ] I have run the [`pre-commit` checks](https://pre-commit.com/) with `./isaaclab.sh --format` - [ ] I have made corresponding changes to the documentation - [ ] 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 - [ ] I have added my name to the `CONTRIBUTORS.md` or my name already exists there <!-- As you go through the checklist above, you can mark something as done by putting an x character in it For example, - [x] I have done this task - [ ] I have not done this task -->
-
Maurice Rahme authored
# Description Fix the logic in `joint_effort_out_of_limit()`; this function aims to detect effort limit violations by comparing the desired (computed) effort against the applied effort, noting that the simulator clips applied torque to the limit assign on creation. Originally, the logic was written such that if the applied and computed torques are equal, then a violation has occurred. However, this is wrong as shown by the below example: ```python from isaaclab.managers import SceneEntityCfg from isaaclab.envs.mdp.terminations import joint_effort_out_of_limit env = ... # any ManagerBasedRLEnv with an Articulation named "robot" cfg = SceneEntityCfg(name="robot", joint_ids=[0]) # single joint for clarity art = env.scene["robot"] # Case A: no clipping (should be False but returns True now) art.data.computed_torque[:] = 0.0 art.data.applied_torque[:] = 0.0 assert joint_effort_out_of_limit(env, cfg).item() is False # CURRENT: True (bug) # Case B: clipping (should be True but returns False now) art.data.computed_torque[:] = 100.0 art.data.applied_torque[:] = 50.0 # pretend actuator clipped to ±50 assert joint_effort_out_of_limit(env, cfg).item() is True # CURRENT: False (bug) ``` The solution is hence simply to flip the limit detection logic. Fixes #3155 ## Type of change - Bug fix (non-breaking change which fixes an issue) ## Checklist - [x] I have run the [`pre-commit` checks](https://pre-commit.com/) with `./isaaclab.sh --format` - [x] I have made corresponding changes to the documentation (N/A) - [x] My changes generate no new warnings - [x] I have added tests that prove my fix is effective or that my feature works - [x] I have updated the changelog and the corresponding version in the extension's `config/extension.toml` file (N/A) - [x] I have added my name to the `CONTRIBUTORS.md` or my name already exists there
-
Soowan Park authored
# Description We found that performance overhead varies depending on the profiler backend, even when profiling is not enabled. For optimization, we changed the default profiler backend to NVTX. Depending on the test case, we observed a performance improvement of about 1–12%. ## Type of change - Bug fix (non-breaking change which fixes an issue) ## Checklist - [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 - [ ] I have added my name to the `CONTRIBUTORS.md` or my name already exists there - [x] I have done this task
-
- 21 Aug, 2025 1 commit
-
-
Kelly Guo authored
# Description A new dependency was picked up by the license checker that has an UNKNOWN license listed. This change adds the license to the exceptions list since it has a MIT license and adds an acknowledgement to the license header. ## Type of change - This change requires a documentation update ## Checklist - [x] I have run the [`pre-commit` checks](https://pre-commit.com/) with `./isaaclab.sh --format` - [x] 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 - [ ] I have added my name to the `CONTRIBUTORS.md` or my name already exists there <!-- As you go through the checklist above, you can mark something as done by putting an x character in it For example, - [x] I have done this task - [ ] I have not done this task -->
-
- 20 Aug, 2025 1 commit
-
-
matthewtrepte authored
# Description <!-- Thank you for your interest in sending a pull request. Please make sure to check the contribution guidelines. Link: https://isaac-sim.github.io/IsaacLab/main/source/refs/contributing.html --> For some simple cases, disabling rate limit in the runloop can help improve performance in headless mode. Follow PR to https://github.com/isaac-sim/IsaacLab/pull/3089, which only updated it for rendering headless app. However, this also impacts non-rendering workflows, so adding it to the default headless app as well. <!-- As a practice, it is recommended to open an issue to have discussions on the proposed pull request. This makes it easier for the community to keep track of what is being developed or added, and if a given feature is demanded by more than one party. --> ## Type of change <!-- As you go through the list, delete the ones that are not applicable. --> - Bug fix (non-breaking change which fixes an issue) ## Checklist - [x] I have run the [`pre-commit` checks](https://pre-commit.com/) with `./isaaclab.sh --format` - [x] 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 - [ ] I have added my name to the `CONTRIBUTORS.md` or my name already exists there <!-- As you go through the checklist above, you can mark something as done by putting an x character in it For example, - [x] I have done this task - [ ] I have not done this task --> --------- Signed-off-by:
matthewtrepte <mtrepte@nvidia.com>
-
- 19 Aug, 2025 1 commit
-
-
Antoine RICHARD authored
# Description Experimental branch to generate observations and actions descriptions from managed environments. ## Type of change - New feature (non-breaking change which adds functionality) - This change requires a documentation update ## Checklist - [x] I have run the [`pre-commit` checks](https://pre-commit.com/) with `./isaaclab.sh --format` - [x] 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:
ooctipus <zhengyuz@nvidia.com>
Co-authored-by:
ooctipus <zhengyuz@nvidia.com>
-
- 18 Aug, 2025 1 commit
-
-
Kelly Guo authored
# Description For some simple cases, disabling rate limit in the runloop can help improve performance for rendering. Nevertheless, we should not need to enable rate limit in headless cases. ## Type of change <!-- As you go through the list, delete the ones that are not applicable. --> - Bug fix (non-breaking change which fixes an issue) - New feature (non-breaking change which adds functionality) ## Checklist - [x] I have run the [`pre-commit` checks](https://pre-commit.com/) with `./isaaclab.sh --format` - [x] 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 - [ ] I have added my name to the `CONTRIBUTORS.md` or my name already exists there <!-- As you go through the checklist above, you can mark something as done by putting an x character in it For example, - [x] I have done this task - [ ] I have not done this task --> Co-authored-by:
ooctipus <zhengyuz@nvidia.com>
-
- 17 Aug, 2025 1 commit
-
-
Louis LE LAY authored
# Description While working on a task I made a tiny typo in the scale randomization range for the stiffness-gain parameter. The robot still spawned, but its behaviour was utterly bizarre. It only took a minute to spot the mistake, yet it made me realize we have no guard-rails for this sort of edge case. This PR introduces a lightweight check that verifies, when operation=="scale", the lower bound is non-negative and the upper bound is not smaller than the lower one. Right now I cover the most common parameters (stiffness, damping, mass, tendon gains, etc.), basically anything that must stay positive to make physical sense. If you’d like the same safeguard applied to other parameter types just let me know and I’ll happily extend the patch. ## Type of change - Bug fix (non-breaking change which fixes an issue) - New feature (non-breaking change which adds functionality) ## Checklist - [x] I have run the [`pre-commit` checks](https://pre-commit.com/) with `./isaaclab.sh --format` - [x] I have made corresponding changes to the documentation - [x] My changes generate no new warnings - [x] 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:
Louis LE LAY <le.lay.louis@gmail.com>
Signed-off-by:
ooctipus <zhengyuz@nvidia.com>
Co-authored-by:
louislelay <louislelay@pal-robotics.com>
Co-authored-by:
ooctipus <zhengyuz@nvidia.com>
-
- 16 Aug, 2025 3 commits
-
-
Pascal Roth authored
# Description Updates CodeOwners to be more fine-grained ## Type of change - Bug fix (non-breaking change which fixes an issue) ## Checklist - [ ] I have run the [`pre-commit` checks](https://pre-commit.com/) with `./isaaclab.sh --format` - [ ] I have made corresponding changes to the documentation - [ ] 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 - [ ] I have added my name to the `CONTRIBUTORS.md` or my name already exists there --------- Co-authored-by:
ooctipus <zhengyuz@nvidia.com> -
Mayank Mittal authored
# Description Some of the information in FAQ is now outdated since Isaac Sim is open-sourced. This MR removes that part but also adds more explaination on the goals of Isaac Lab. ## Type of change - This change requires a documentation update ## Checklist - [x] I have run the [`pre-commit` checks](https://pre-commit.com/) with `./isaaclab.sh --format` - [x] 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:
Mayank Mittal <12863862+Mayankm96@users.noreply.github.com>
Co-authored-by:
Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by:
ooctipus <zhengyuz@nvidia.com> -
Mayank Mittal authored
# Description This MR introduces the following: * An `agent` argument to all scripts to allow selecting different entry points (each then get resolved to their respective settings file). * Symmetry function for ANYmal robot for the locomotion task and cartpole balancing task * Documentation on how to configure RL training agent using gym resgistry Fixes #2835 ## Type of change - New feature (non-breaking change which adds functionality) - This change requires a documentation update ## Screenshots ### Cartpole ```bash # without symmetry ./isaaclab.sh -p scripts/reinforcement_learning/rsl_rl/train.py --task Isaac-Cartpole-v0 --headless --agent rsl_rl_with_symmetry_cfg_entry_point --run_name ppo_with_no_symmetry agent.algorithm.symmetry_cfg.use_data_augmentation=false # with symmetry ./isaaclab.sh -p scripts/reinforcement_learning/rsl_rl/train.py --task Isaac-Cartpole-v0--headless --agent rsl_rl_with_symmetry_cfg_entry_point --run_name ppo_with_symmetry_data_augmentation agent.algorithm.symmetry_cfg.use_data_augmentation=true ``` | Isaac-Cartpole-v0 (pink w/o symmetry, blue w/ symmetry) | | ------ | | <img width="823" height="421" alt="image" src="https://github.com/user-attachments/assets/9c33db99-0d79-4c1d-b437-e01275d613b5" /> | ### Locomotion ```bash # without symmetry ./isaaclab.sh -p scripts/reinforcement_learning/rsl_rl/train.py --task Isaac-Velocity-Rough-Anymal-D-v0 --headless --agent rsl_rl_with_symmetry_cfg_entry_point --run_name ppo_with_no_symmetry agent.algorithm.symmetry_cfg.use_data_augmentation=false # with symmetry ./isaaclab.sh -p scripts/reinforcement_learning/rsl_rl/train.py --task Isaac-Velocity-Rough-Anymal-D-v0 --headless --agent rsl_rl_with_symmetry_cfg_entry_point --run_name ppo_with_symmetry_data_augmentation agent.algorithm.symmetry_cfg.use_data_augmentation=true ``` | Isaac-Velocity-Rough-Anymal-D-v0 (green w/o symmetry, purple w/ symmetry) | | ------ | | <img width="1241" height="414" alt="image" src="https://github.com/user-attachments/assets/625c125d-db9f-4006-9a62-0d55701a9407" /> | ## Checklist - [x] I have run the [`pre-commit` checks](https://pre-commit.com/) with `./isaaclab.sh --format` - [x] 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 --------- Co-authored-by:
Kelly Guo <kellyg@nvidia.com>
Co-authored-by:
ooctipus <zhengyuz@nvidia.com>
-
- 15 Aug, 2025 2 commits
-
-
Ziqi Fan authored
Changes the expected input of checkpoint in rsl-rl to an absolute path in documentation to avoid ambiguity (#3151) # Description If the user directly enters model_xxx.pt, an error will be reported. The user should be guided to enter a path ## Type of change - This change requires a documentation update ## Checklist - [x] I have run the [`pre-commit` checks](https://pre-commit.com/) with `./isaaclab.sh --format` - [x] 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 Co-authored-by:
ooctipus <zhengyuz@nvidia.com> -
James Tigue authored
# Description This PR: - Adds ContactSensorCfg.track_contact_points to toggle tracking of contact point locations between sensor bodies and filtered bodies. - Adds ContactSensorCfg.max_contact_data_per_prim to configure the maximum amount of contacts per sensor body. - Adds ContactSensorData.contact_pos_w data field for tracking contact point locations. Fixes # (issue) <!-- As a practice, it is recommended to open an issue to have discussions on the proposed pull request. This makes it easier for the community to keep track of what is being developed or added, and if a given feature is demanded by more than one party. --> ## Type of change <!-- As you go through the list, delete the ones that are not applicable. --> - New feature (non-breaking change which adds functionality) ## Checklist - [ ] I have run the [`pre-commit` checks](https://pre-commit.com/) with `./isaaclab.sh --format` - [ ] I have made corresponding changes to the documentation - [ ] 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 - [ ] I have added my name to the `CONTRIBUTORS.md` or my name already exists there --------- Signed-off-by:
James Tigue <166445701+jtigue-bdai@users.noreply.github.com>
Signed-off-by:
Kelly Guo <kellyg@nvidia.com>
Signed-off-by:
ooctipus <zhengyuz@nvidia.com>
Co-authored-by:
Ashwin Khadke <133695616+akhadke-bdai@users.noreply.github.com>
Co-authored-by:
Kelly Guo <kellyg@nvidia.com>
Co-authored-by:
ooctipus <zhengyuz@nvidia.com>
-
- 14 Aug, 2025 3 commits
-
-
Emily Sturman authored
# Description Fixes the IndexError caused by simultaneously indexing env_ids and joint_ids in `reset_joints_by_scale` and `reset_joints_by_offset`. Fixes # [2948](https://github.com/isaac-sim/IsaacLab/issues/2948) ## Type of change <!-- As you go through the list, delete the ones that are not applicable. --> - Bug fix (non-breaking change which fixes an issue) ## Checklist - [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 - [x] 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:
Kelly Guo <kellyg@nvidia.com>
Signed-off-by:
Emily Sturman <emily@sturman.org>
Signed-off-by:
ooctipus <zhengyuz@nvidia.com>
Co-authored-by:
Kelly Guo <kellyg@nvidia.com>
Co-authored-by:
Mayank Mittal <12863862+Mayankm96@users.noreply.github.com>
Co-authored-by:
ooctipus <zhengyuz@nvidia.com> -
matthewtrepte authored
# Description <!-- Thank you for your interest in sending a pull request. Please make sure to check the contribution guidelines. Link: https://isaac-sim.github.io/IsaacLab/main/source/refs/contributing.html --> Update rendering preset mode pathing to be compatible when an experience file is set. <!-- As a practice, it is recommended to open an issue to have discussions on the proposed pull request. This makes it easier for the community to keep track of what is being developed or added, and if a given feature is demanded by more than one party. --> ## Type of change <!-- As you go through the list, delete the ones that are not applicable. --> - Bug fix (non-breaking change which fixes an issue) ## Screenshots Please attach before and after screenshots of the change if applicable. <!-- Example: | Before | After | | ------ | ----- | | _gif/png before_ | _gif/png after_ | To upload images to a PR -- simply drag and drop an image while in edit mode and it should upload the image directly. You can then paste that source into the above before/after sections. --> ## Checklist - [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 - [x] I have updated the changelog and the corresponding version in the extension's `config/extension.toml` file - [ ] I have added my name to the `CONTRIBUTORS.md` or my name already exists there <!-- As you go through the checklist above, you can mark something as done by putting an x character in it For example, - [x] I have done this task - [ ] I have not done this task --> Signed-off-by:
ooctipus <zhengyuz@nvidia.com>
Co-authored-by:
ooctipus <zhengyuz@nvidia.com> -
ooctipus authored
Fixes Termination Manager logging to report aggregated percentage of environments done due to each term. (#3107) # Description Currently Termination Manager write current step's done count for each term if reset is detected. This leads to two problem. 1. User sees different counts just by varying num_envs 2. the count can be over-count or under-count depending on when reset happens, as pointed out in #2977 (Thanks, @Kyu3224) The cause of the bug is because we are reporting current step status into a buffer that suppose to record episodic done. So instead of write the entire buffer base on current value, we ask the update to respect the non-reseting environment's old value, and instead of reporting count, we report percentage of environment that was done due to the particular term. Test on Isaac-Velocity-Rough-Anymal-C-v0 Before fix: <img width="786" height="323" alt="Screenshot from 2025-08-06 22-16-20" src="https://github.com/user-attachments/assets/4838d612-7f0e-4232-a07e-688b547e91db" /> Red: num_envs = 4096, Orange: num_envs = 1024 After fix: <img width="786" height="323" alt="Screenshot from 2025-08-06 22-16-12" src="https://github.com/user-attachments/assets/e6e55c21-17ed-42ca-8d94-a19d08611f86" /> Red: num_envs = 4096, Orange: num_envs = 1024 Note that curve of the same color ran on same seed, and curves matched exactly, the only difference is the data gets reported in termination. The percentage version is a lot more clear in conveying how agent currently fails, and how much percentage of agent fails, and shows that increasing num_envs to 4096 helps improve agent avoiding termination by `base_contact` much quicker than num_envs=1024. Such message is a bit hard to tell in first image. <!-- Example: | Before | After | | ------ | ----- | | _gif/png before_ | _gif/png after_ | To upload images to a PR -- simply drag and drop an image while in edit mode and it should upload the image directly. You can then paste that source into the above before/after sections. --> ## Checklist - [x] I have run the [`pre-commit` checks](https://pre-commit.com/) with `./isaaclab.sh --format` - [ ] I have made corresponding changes to the documentation - [ ] My changes generate no new warnings - [ ] I have added tests that prove my fix is effective or that my feature works - [x] 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 <!-- As you go through the checklist above, you can mark something as done by putting an x character in it For example, - [x] I have done this task - [ ] I have not done this task -->
-
- 13 Aug, 2025 1 commit
-
-
ooctipus authored
# Description Current is_isaacsim_version_4_5() sometimes mis-detects 5.0 when running 4.5 (e.g., Docker/symlinked installs), causing the conda bootstrap to pick Python 3.11 instead of 3.10. This pr ensures determining the isaacsim version following below steps 1. Read ${ISAACLAB_PATH}/_isaac_sim/VERSION (symlink/binary install). 2. If unknown, import isaacsim and read ../../VERSION next to the package. (pip installation) 3. If still unknown, fall back to importlib.metadata.version("isaacsim"). I tested locally to work with 4.5 pip or binary install. <!-- As you go through the list, delete the ones that are not applicable. --> - Bug fix (non-breaking change which fixes an issue) ## Screenshots Please attach before and after screenshots of the change if applicable. <!-- Example: | Before | After | | ------ | ----- | | _gif/png before_ | _gif/png after_ | To upload images to a PR -- simply drag and drop an image while in edit mode and it should upload the image directly. You can then paste that source into the above before/after sections. --> ## Checklist - [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 - [x] 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 <!-- As you go through the checklist above, you can mark something as done by putting an x character in it For example, - [x] I have done this task - [ ] I have not done this task -->
-
- 08 Aug, 2025 4 commits
-
-
Ashwin Varghese Kuruttukulam authored
# Bugfix Previously, self._data.joint_dynamic_friction_coeff was being set for all environments, regardless of the intended targets. This behavior is incorrect — the friction coefficient should only be set for the specified physx_env_ids. Co-authored-by:
ooctipus <zhengyuz@nvidia.com>
Co-authored-by:
Kelly Guo <kellyg@nvidia.com> -
Hunter Hansen authored
# Description This adds improved readout from the subprocesses in `install_deps.py`. It will now print the output instead of printing everything at once when the process is finished. It also has improved error handling. This will help users to have a better understanding of their build process and more easily resolve issues. ## Type of change <!-- As you go through the list, delete the ones that are not applicable. --> - New feature (non-breaking change which adds functionality) ## Checklist - [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 <!-- As you go through the checklist above, you can mark something as done by putting an x character in it For example, - [x] I have done this task - [ ] I have not done this task --> --------- Signed-off-by:
Hunter Hansen <50837800+hhansen-bdai@users.noreply.github.com>
Signed-off-by:
Kelly Guo <kellyg@nvidia.com>
Co-authored-by:
Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by:
Kelly Guo <kellyg@nvidia.com> -
Kelly Guo authored
# Description Updates the release notes docs with the latest release notes for 2.2 and some other minor documentation updates. Also fixes new issues with the license checker to add additional exceptions. Also using this change to trigger the documentation update for 2.2. ## Type of change - This change requires a documentation update ## Checklist - [x] I have run the [`pre-commit` checks](https://pre-commit.com/) with `./isaaclab.sh --format` - [x] 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 - [ ] I have added my name to the `CONTRIBUTORS.md` or my name already exists there <!-- As you go through the checklist above, you can mark something as done by putting an x character in it For example, - [x] I have done this task - [ ] I have not done this task -->
-
Alexander Poddubny authored
# Description 1. Changed test reporter for compatibility pipeline to generate HTML report 2. Increased the number of reported logs for failed test cases to 50 (max) 3. Set execute permissions in the Dockerfile on the isaaclab.sh - sometimes the post-merge pipeline fails due to permissions 4. Updated the default Isaac SIM version
-