- 29 Aug, 2024 1 commit
-
-
Kelly Guo authored
# Description * Fix tested version in docs * Add sb3 example for Hydra ## 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
-
- 24 Aug, 2024 1 commit
-
-
James Smith authored
# Description This PR adds in a [CODEOWNERS](https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/about-code-owners) file so that we can better handle the review request process. ## Type of change - 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 --------- Signed-off-by:
Mayank Mittal <12863862+Mayankm96@users.noreply.github.com>
Co-authored-by:
Mayank Mittal <mittalma@leggedrobotics.com>
Co-authored-by:
Mayank Mittal <12863862+Mayankm96@users.noreply.github.com>
Co-authored-by:
jtigue-bdai <166445701+jtigue-bdai@users.noreply.github.com>
-
- 23 Aug, 2024 1 commit
-
-
Kelly Guo authored
# Description When cameras are enabled, a default render product always gets created for the viewport, which brings additional overhead for rendering the viewport at each step. However, when running in headless mode, we may not need to render the viewport if we are not recording or livestreaming. This change disables viewport rendering in the headless cases where recording and livestreaming are not required. ## 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 - [ ] 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, 2024 2 commits
-
-
Kelly Guo authored
# Description When configclass dicts are nested inside lists, the list is treated as an Iterable object and assigned directly to the outer configclass when updating configclass data with dicts. This overwrites the configclass object in the list with a dict object and causes undesired behavior. This change checks for nested dictionaries inside Iterables and updates the values inside the dictionary individually without overwiting the full Iterable. Fixes https://github.com/isaac-sim/IsaacLab/issues/843 ## 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 - [ ] 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
-
Masoud Moghani authored
# Description When installing Isaac Lab in conda environment, numpy version 2.0 gets installed, which breaks all the scripts from running properly. This MR fixes compatibility issues with NumPy by setting its installation version to less than 2.0. ## 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 - [x] I have added my name to the `CONTRIBUTORS.md` or my name already exists there
-
- 21 Aug, 2024 2 commits
-
-
Michael Gussert authored
# Description I added images for the tutorials and modified the docs to include them ## 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:
Michael Gussert <michael@gussert.com>
Signed-off-by:
Kelly Guo <kellyg@nvidia.com>
Co-authored-by:
Kelly Guo <kellyg@nvidia.com> -
Masoud Moghani authored
# Description This MR adds deformable object API to assets in the core framework. The class creates two physics views: one for the deformable object and the other one for the deformable material. Based on these, the users can set and get different nodal information from the solver, as well as randomize deformable material properties. The MR also adds some basic tests and a script in the tutorial that showcases the deformable object. ## Type of change - New feature (non-breaking change which adds functionality) - This change requires a documentation update ## Screenshots Output from: ```bash ./isaaclab.sh -p source/standalone/tutorials/01_assets/run_deformable_object.py ``` https://github.com/user-attachments/assets/9265f4d4-bebf-41b4-9d73-35c558f47a15 ## 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:
Mayank Mittal <12863862+Mayankm96@users.noreply.github.com>
Co-authored-by:
Mayank Mittal <mittalma@leggedrobotics.com>
-
- 19 Aug, 2024 1 commit
-
-
StrainFlow authored
# Description In the verification step for Isaac Sim installed from binaries, if I set the `ISAACSIM_PYTHON_EXE` variable as specified and print it, there are extra quotes, so the later steps don't work. This MR fixes the environment variable in Windows verification. ## Type of change - 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 Signed-off-by:
StrainFlow <107502145+StrainFlow@users.noreply.github.com>
-
- 17 Aug, 2024 3 commits
-
-
jtigue-bdai authored
# Description The goal of this PR is to introduce expanded functionality to observation corruption/realism. This is done by introducing the concept of configurable modifiers. This creates an architecture that can apply a list of custom modifiers to observation data giving flexibility to order of operations. Existing modifiers for clip, scale, bias, and noise can be used. This does so without changing the current functionality. This PR also includes two new modifiers that are stateful (i.e. output is dependent on previous state) that utilize a Callable class. They are a `DigitalFilter` and an `Integrator`. Summary of Additions: - Adds `ModifierCfg` and `Modifier` - Adds example modifiers functions for bias, scale, clip, noise - Adds example modifiers classes: DigitalFilter, Integrator - Adds functionality to `ObservationManager` to prepare and handle modifiers of observation data - Adds unit tests for modifiers in test_modifiers.py - Adds test to test_observation_manager.py to test integration with modifiers ## 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 - [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:
jtigue-bdai <166445701+jtigue-bdai@users.noreply.github.com>
Co-authored-by:
Mayank Mittal <mittalma@leggedrobotics.com> -
Pascal Roth authored
# Description Currently, for multiple runs, the code is pushed to the same location on the cluster. When the runs are not directly started (i.e., are in the queue), the code will be replaced each time a new job is submitted. This PR creates a new copy of the code for each job to ensure that the intended version is used for the submitted job. ## Type of change - 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 run all the tests with `./isaaclab.sh --test` and they pass - [ ] 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
-
David Hoeller authored
# Description This MR adds utilities to enable the hydra configuration system. Using the `train.py` scripts, the user can now change any parameter in the environment or agent configs from command line inputs, for example: ``` python source/standalone/workflows/rsl_rl/train.py --task Isaac-Cartpole-v0 --headless env.actions.joint_effort.scale=10.0 ``` ## Type of change - 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:
David Hoeller <dhoeller@nvidia.com>
Co-authored-by:
Mayank Mittal <12863862+Mayankm96@users.noreply.github.com>
Co-authored-by:
Kelly Guo <kellyg@nvidia.com>
-
- 15 Aug, 2024 2 commits
-
-
Mayank Mittal authored
# Description Mainly documentation fixes and ensuring the code quality remains consistent. Also adds the `container.sh` script which calls the python script for compatibility reasons. We can remove it in later releases. ## Type of change - 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
-
Mayank Mittal authored
# Description Noticed that we were checking for function arguments inside a for-loop, which isn't necessary. Moved this check outside to make it simpler to read the code. Also noticed a small corner case in the event manager when reset is called and `env_ids` is None. In that case, it would bypass the check for min steps between reset and directly apply the term to the environment. I am not sure if that was intentional. if so, I can revert the behavior. ## 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
-
- 14 Aug, 2024 4 commits
-
-
Pascal Roth authored
# Description Fixes cluster setting after merge of [python conversion](https://github.com/isaac-sim/IsaacLab/commit/f565c33d7716db1be813b30ddbcf9321712fc497) ## 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 - [x] I have added my name to the `CONTRIBUTORS.md` or my name already exists there
-
Mayank Mittal authored
# Description Long lists become really hard to read and understand. This MR switches them to tables instead. ## Type of change - This change requires a documentation update ## Screenshots  ## 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
-
Hunter Hansen authored
# Description This PR recreates the capabilities initially made by @farbod-farshidian [here](https://github.com/isaac-sim/IsaacLab/pull/455) for the new Pythonized-container interface. ## Type of change <!-- As you go through the list, delete the ones that are not applicable. --> - 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 run all the tests with `./isaaclab.sh --test` and they pass - [ ] 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:
James Smith <142246516+jsmith-bdai@users.noreply.github.com>
Co-authored-by:
James Smith <142246516+jsmith-bdai@users.noreply.github.com>
Co-authored-by:
David Hoeller <dhoeller@nvidia.com> -
Hunter Hansen authored
This PR replicates the capabilities of `container.sh` in Python, and breaks its capabilities up into several files under a new directory `/docker/isaaclab_container_utils`, as well as a superior interface in `container.py`. The intention of this change is to make our container-mediating code more easily readable, debuggable, and modifiable. It is also done in the hopes that it can be more easily distributed as we see a desire from users to [compose and modify our setup](https://github.com/isaac-sim/IsaacLab/pull/455). It also has the additional benefit of needing fewer sudo installs because of Python's native yaml handling. The central class, `IsaacLabContainerInterface`, contains a lot of the original utility of the script, and several of the current `container.py` scripts options simply configure it and call a method. @pascal-roth `apptainer_utils.py` and the `./container.py job/push` logic are separated out, I'm curious what you think of this delineation. I also haven't been able to fully test that end of things as I don't have a cluster to use, though I did verify that it worked to the extent I could. I will update the docs when I have received approval <!-- As you go through the list, delete the ones that are not applicable. --> - Breaking change (fix or feature that would cause existing functionality to not work as expected) - This change requires a documentation update - [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 run all the tests with `./isaaclab.sh --test` and they pass - [ ] 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:
Hunter Hansen <50837800+hhansen-bdai@users.noreply.github.com>
Signed-off-by:
James Smith <142246516+jsmith-bdai@users.noreply.github.com>
Co-authored-by:
James Smith <142246516+jsmith-bdai@users.noreply.github.com>
Co-authored-by:
David Hoeller <dhoeller@nvidia.com>
-
- 13 Aug, 2024 1 commit
-
-
sizsJEon authored
# Description The rl_games train.py script did not include the checkpoint and sigma. (it was none) I have added these features and verified their operation. - 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
-
- 07 Aug, 2024 3 commits
-
-
xavier authored
# Description Adds the attribute `preserve_order` in `JointActionCfg`, to choose whether joint-related actions should keep the same order as how the `joint_names` are listed in the config. Fixes #786 ## Type of change - New feature ## 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 - [ ] 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
-
Masoud Moghani authored
# Description This MR resolves a naming discrepancy in the rsl-rl agent configuration files across multiple tasks. By aligning all tasks to use the `rsl_rl_ppo_cfg` naming convention, it ensures uniformity and clarity in configuration file names. ## 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 - [x] I have added my name to the `CONTRIBUTORS.md` or my name already exists there
-
David Hoeller authored
# Description Since the move to the ``--device`` cli to set the device to run the simulation on, this clashes with ``--device`` for the teleoperation environments. This MR renames ``--device`` to ``--teleop_device`` for these examples. ## 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 - [ ] 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
-
- 06 Aug, 2024 3 commits
-
-
David Hoeller authored
# Description Fixes importing rsl_rl configs ## 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 - [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
-
David Hoeller authored
# Description Previously, it was not possible to run training scripts with rl_games, skrl, sb3 when rsl_rl was not installed. Fixes #749 ## 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 - [x] I have added my name to the `CONTRIBUTORS.md` or my name already exists there
-
David Hoeller authored
# Description Before setting up the device was tedious and redundant, and you would see code like: ```python sim_utils.SimulationCfg(device="cpu", use_gpu_pipeline=False, dt=0.01, physx=sim_utils.PhysxCfg(use_gpu=False)) ``` With this MR, we simply set the desired device, and all the parameters, such as `use_gpu_pipeline` and `use_gpu` propagate automatically, resulting in the following: ```python sim_utils.SimulationCfg(device="cpu", dt=0.01, physx=sim_utils.PhysxCfg()) ``` The command line input `--cpu` has been removed in favor of `--device device_name`, where valid options for `device_name` are `cpu` (run on CPU), `cuda` (run on GPU with device id 0), or `cuda:N` (run on GPU with device id N, for example `cuda:0`). ## Type of change - Breaking Change ## 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 - [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
-
- 05 Aug, 2024 1 commit
-
-
Johnson Sun authored
# Description This MR enables users to easily record videos from trained checkpoints, which is useful for sanity checks or creating promotional videos for research papers. Fixes: https://github.com/isaac-sim/IsaacLab/issues/130 ## 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 - [ ] 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
-
- 03 Aug, 2024 3 commits
-
-
Mayank Mittal authored
# Description Previously, the observation manager rigidly tried concatenating all the terms along the last dimension at construction. However, when one has hybrid-dimensional terms inside the observation group, this operation would fail, leading to an error. In principle, the concatenation should only happen when users set the attribute `concatenate_terms` to True inside the group settings. This MR lifts the above operation and adds a proper error when users try to concatenate observation terms of different shapes. They must set the concatenation flag to False if they want a "hybrid" observation group. Fixes #768 ## Type of change - 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
-
Kelly Guo authored
# Description This change refactors the shadow hand environment into a general in-hand manipulation environment and extends it with the Allegro hand. ## Type of change - 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 - [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
-
Kelly Guo authored
# Description - Adds `min_step_count_between_reset` for Event terms to specify minimum randomization frequency for reset randomizations - Switch to numpy for material bucketing logic - Updates SH Config with DR ## 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 - [ ] I have added tests that prove my fix is effective or that my feature works - [x] I have run all the tests with `./isaaclab.sh --test` and they pass - [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
-
- 02 Aug, 2024 12 commits
-
-
James Smith authored
# Description NOTE: This is a redo of https://github.com/isaac-orbit/IsaacLab/pull/524, only rebased onto new main and with changes to work with Isaac Lab. Copied from other PR: Modifies container.sh and x11.yaml to automatically check if a tmp.***.xauth file has a stale MIT-MAGIC-COOKIE-1, and refreshes it if so. In the instance where an xsession ends after a `tmp.***.xauth` file has been created, but the system is not rebooted (thus deleting `tmp.***.xauth`) then attempting to use the container causes errors. This is a feature to correct for this upon `container.sh enter` by: 1. Mounting an entire `/tmp` subdirectory so that we can swap files 2. Checking for stale cookies by comparing against the current user's cookie 3. If stale, replacing the old `tmp.***.xauth` with a new `tmp.***.xauth` of the same name but a new cookie, in order to preserve functionality of envars. ## 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 `./orbit.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 run all the tests with `./orbit.sh --test` and they pass - [ ] 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:
James Smith <142246516+jsmith-bdai@users.noreply.github.com>
Co-authored-by:
Hunter Hansen <50837800+hhansen-bdai@users.noreply.github.com> -
Mayank Mittal authored
# Description Since we override a lot of the functions from RigidObject inside the Articulation class, we don't need to rely on inheritance anymore. Duplicacy in the code makes it easier to understand the two classes' functionalities without severely added overhead from the maintenance side. Moreover, conceptually, it can be motivated that the two are independent concepts. This MR decouples the rigid object and articulation concepts in the framework. ## Type of change - Breaking change (fix or feature that would cause existing functionality to not work as expected) - 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 - [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 --------- Co-authored-by:
David Hoeller <dhoeller@nvidia.com> -
Kelly Guo authored
# Description - Removes unneeded extensions from headless app file - Limits CPU threads when running multi-GPU training to minimize thread context switching - Apply multi GPU renderer setting to avoid unnecessary GPU context initialization - Decreases 8k SH startup time on 8 GPUs from > 5 min to 1 min - Adds benchmarking scripts ## 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 - [ ] 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:
Alexander <143108850+nv-apoddubny@users.noreply.github.com>
Co-authored-by:
David Hoeller <dhoeller@nvidia.com> -
James Smith authored
# Description Add a cli argument `--extension` to run only unit tests for a selected extension. As an example: ```bash isaaclab -t --extension omni.isaac.lab_assets # Only runs tests under omni.isaac.lab_assets ``` Fixes #752 ## Type of change - 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 - [x] I have added my name to the `CONTRIBUTORS.md` or my name already exists there
-
Mayank Mittal authored
# Description Earlier the README had hard-coded values for "passing" of pre-commit and docs. This changes it to use the GitHub actions instead. ## 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
-
Pascal Roth authored
# Description Adds a parameter `border_height` to `TerrainGeneratorCfg` to be able to control the height of the generator border instead of having a hard-coded value of 1.0m. ## Type of change - 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 - [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
-
Mayank Mittal authored
# Description There was a slight inconsistency-- we called `unitree_a1`, `unitree_go1`, and at the same time `h1` and `g1`. This MR fixes this inconsistency and adds some more docs. ## 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` - [ ] 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
-
Mayank Mittal authored
# Description The caching of terrains with the terrain generator did not work as expected even when the curriculum was enabled (which should have yielded the same terrain every time). On deeper investigation, it appears that the terrain generator samples a range when using the curriculum to avoid having the same sub-terrain in columns of the same terrain type. Thus, the terrain `difficulty` was stochastic between runs, which affected the caching of the terrain. This MR fixes the following: - Adds documentation of the above behavior - Modifies the generator to use `np.random.Generator` (RNG) instead of setting the seed globally - Adds a warning if the user wants to cache the terrain and the seed is not set - Adds tests to check caching works for the terrain generator with and without curriculum Note: Right now, complete determinism (i.e., the same terrain is generated) is only possible when seed is set and caching is enabled. If caching is disabled, it is possible that the terrain may have small differences. To fix this, the RNG needs to be propagated to all the generation functions as well. I leave that as a future TODO. ## Type of change - 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 - [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
-
Mayank Mittal authored
# Description This MR adds docstrings in articulation and rigid object data to clarify the exact frames they are reported in. This was previously causing confusion as PhysX reports the transforms in actor frame and velocities in CoM frame. Right now, there is no mechanism to get all this data in the same frame, so the best we can do is document behavior. ## 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
-
Mayank Mittal authored
# Description This MR renames the action term names inside the manager-based lift-manipulation task. Earlier, they were called ``body_joint_pos`` and ``gripper_joint_pos``. This made it confusing since they were also being set to IK instead of joint position action terms. Now, they are called ``arm_action`` and ``gripper_action``. This makes it consistent with other manipulation tasks. ## Type of change - Breaking change (fix or feature that would cause existing functionality to not work as expected) - 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 - [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
-
Masoud Moghani authored
# Description This MR removes the additional translation and orientation applied when creating the mesh prim under the parent Xform prim. ## 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 - [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
-
Mayank Mittal authored
# Description Previously, we used finite differencing to compute rigid body accelerations. Since Isaac Sim 4.0, we can obtain the accelerations directly from PhysX. This MR removes the need for finite differencing. ## 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 - [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
-