1. 07 Feb, 2024 3 commits
    • AutonomousHansen's avatar
      Specifies the torch version dependency explicitly as 2.0.1 (#376) · 37963a3d
      AutonomousHansen authored
      # Description
      
      This MR pins the torch version to 2.0.1 in installing the `orbit` and
      `orbit_tasks` extensions. This helps fix the torch and typing_extensions
      version incompatibility reported in #373.
      
      Previously, torch version 2.2 was getting downloaded on a fresh
      installation, which caused an error when trying to import it after
      launching Isaac Sim: `NameError: name '_C' is not defined. Did you mean:
      '_'?`
      
      Fixes #373 
      
      ## 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
      `./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
      - [x] I have run all the tests with `./orbit.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
      37963a3d
    • Mayank Mittal's avatar
      Fixes handling of time-out signal in RSL-RL and RL-Games wrapper (#375) · af4e8014
      Mayank Mittal authored
      # Description
      
      On termination of an episode, three conditions arise:
      
      1. **bad** terminations (terminated dones): the agent gets a termination
      penalty
      2. **timeout** terminations (truncated dones):
      * infinite-horizon: bootstrapping by the agent based on terminal state
          * finite-horizon: no penalty or bootstrapping
      
      Currently, we have not handled the last case, which leads to issues when
      training RL tasks with a finite horizon (for instance, Nikita's agile
      locomotion work).
      
      This MR adds a flag to the RLTaskEnvCfg called `is_finite_horizon` that
      helps deal with this case. The flag is consumed by the env wrappers to
      decide how they want to specifically handle the finite horizon problem.
      
      ## 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
      `./orbit.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 `./orbit.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
      af4e8014
    • AutonomousHansen's avatar
      Set livestream enabled to false unless it is otherwise specified (#374) · 93ec2c60
      AutonomousHansen authored
      From Isaac `2023.1.0-hotfix.1` to `2023.1.1`, `/app/livestream/enabled`
      started to default to True. This breaks our logic w/r/t
      `SimulationContext.render()`, which is now treating a headless instance
      as one that needs full rendering. This causes problems with multiple
      SimulationContext calls, and ultimately this bottoms out with this
      [line](https://github.com/isaac-orbit/orbit/blob/670a8c97db645d50a3ea89b06f7366fc2212fc06/source/extensions/omni.isaac.orbit/omni/isaac/orbit/sim/simulation_context.py#L563)
      sending us into an infinite rendering cycle from which i.e. unittest
      cannot exit.
      
      I fix this problem by setting `/app/livestream/enabled` to explicitly
      False in AppLauncher unless otherwise requested. This corrects the hang.
      
      Ultimately, the answer to this is #238, but in the meantime we need a
      fix or this is a showstopper for unit testing.
      
      Fixes #323
      
      - Bug fix (non-breaking change which fixes an issue)
      
      - [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
      - [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
      93ec2c60
  2. 30 Jan, 2024 2 commits
    • Nikita Rudin's avatar
      Moves height-scan observation offset to its function's argument (#366) · 670a8c97
      Nikita Rudin authored
      # Description
      
      Adds an optional argument to the height-scan obs term defining the
      offset. Previously, it was hardcoded to 0.5.
      
      ## 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
      `./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
      - [ 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
      670a8c97
    • Nikita Rudin's avatar
      Adds approximate torque calculation for implicit actuator (#364) · 882f7c04
      Nikita Rudin authored
      # Description
      
      Adds torque computation to implicit actuators. Previously the reported
      torque was always zero.
      
      ## 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
      `./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
      - [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
      882f7c04
  3. 23 Jan, 2024 2 commits
    • renezurbruegg's avatar
      Fixes missing max raycast distance in RayCaster sensor (#48) · b579c503
      renezurbruegg authored
      # Description
      
      This MR fixes a minor issue where the max raycast distance in the config was ignored
      in the raycaster sensor.
      
      ## 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
      `./orbit.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 `./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
      
      ---------
      Co-authored-by: 's avatarPascal <roth.pascal@outlook.de>
      b579c503
    • Pascal Roth's avatar
      Adds `apptainer` version in the cluster guide docs (#357) · 82258319
      Pascal Roth authored
      After reporting binding issues on the cluster, this fix mentions the
      `apptainer version 1.2.5-1.el7` used in our workflow in the cluster
      documentation.
      
      - Bug fix (non-breaking change which fixes an issue)
      
      - [x] I have run the [`pre-commit` checks](https://pre-commit.com/) with
      `./orbit.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 `./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
      82258319
  4. 22 Jan, 2024 2 commits
  5. 12 Jan, 2024 1 commit
    • AutonomousHansen's avatar
      Comments out X11Forwarding bindings in docker-compose.yaml (#348) · 9c3f65f3
      AutonomousHansen authored
      # Description
      
      We currently have X11 forwarding enabled by default. This is convenient
      for certain users who want graphical displays (including IsaacSim
      itself) to be forwarded from within the container. However, it is also
      partially dependent upon the configuration of the host system and
      requires a valid .Xauthority file which may or may not have been
      created. Because this has introduced difficulties for several of our
      users, I submit that we comment this portion out be default, until a
      solution which allows either a `./container.sh` option, a variable in
      `.env`, or both can be used to conveniently enable this option on an
      affirmative basis.
      
      If we can agree on this change, I will update the documentation
      correspondingly.
      
      ## Type of change
      
      - Breaking change (fix or feature that would cause existing
      functionality to not work as expected)
      
      ## 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 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
      9c3f65f3
  6. 11 Jan, 2024 2 commits
    • jsmith-bdai's avatar
      Fixes source frame indexing in FrameTransfomer sensor (#350) · 620ce2b0
      jsmith-bdai authored
      # Description
      
      The FrameTransformer sensor always assumed that the source frame index
      is 0 in the parsed regex expression. However, a recent issue appeared
      where this is not the case, and it led to a bug due to the hard-coded
      value. This MR fixes the source frame index in the sensor and also adds
      a unit test to check this behavior.
      
      Fixes: https://github.com/NVIDIA-Omniverse/Orbit/issues/170
      
      ## 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
      `./orbit.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 run all the tests with `./orbit.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
      
      ---------
      Co-authored-by: 's avatarMayank Mittal <mittalma@leggedrobotics.com>
      620ce2b0
    • Mayank Mittal's avatar
  7. 10 Jan, 2024 2 commits
  8. 09 Jan, 2024 2 commits
    • Anton Bjørndahl Mortensen's avatar
      Fixes AttributeError in terrain-based position command (#191) · ea711bcf
      Anton Bjørndahl Mortensen authored
      Fixes AttributeError in position_command.py and adds a dummy function to TerrainImporter for sampling target locations.
      
      Fixes #189
      
      - Bug fix (non-breaking change which fixes an issue)
      
      - [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
      - [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
      ea711bcf
    • jsmith-bdai's avatar
      Adds a runner script to execute all tests in the `source` directory (#327) · 51ccd99f
      jsmith-bdai authored
      # Description
      
      This PR adds a script to run all the unit tests (any Python script named
      `test_*.py`) and report on the success rate and timing. It also enables
      the skipping of known breaking tests via tests_to_skip.py. The tests can
      be run via `orbit -t`, and the test output will be printed to both a log
      (`test_results.log`) and to the console.
      
      This can be used for CI/CD but also can be temporarily used to run each
      test without invoking them manually one by one until we have a better
      solution.
      
      A few tests are currently broken; for now, these are added to
      tests_to_skip - I will put follow-up issues + PRs to fix these as soon
      as possible. I recommend we add running tests as a requirement before
      merging PRs until the CI/CD project is complete, too. One way to help
      facilitate this is to have PR authors copy/paste the results at the end
      of the test run to the PR description.
      
      Fixes #309 
      
      ## Type of change
      
      - New feature (non-breaking change which adds functionality)
      
      ## Screenshots
      
      An example test result summary with timeout set to 60 seconds running on
      my workstation:
      
      ```
      ===================
      Test Result Summary
      ===================
      Total: 39
      Passing: 29
      Failing: 0
      Skipped: 8
      Timing Out: 2
      Passing Percentage: 94.87%
      Total Time Elapsed: 0.0h31.0m52.38s
      ```
      
      ## 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
      - [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
      - [ ] I have added my name to the `CONTRIBUTORS.md` or my name already
      exists there
      51ccd99f
  9. 05 Jan, 2024 1 commit
    • Mayank Mittal's avatar
      Updates license year to 2024 (#345) · 963f3045
      Mayank Mittal authored
      # Description
      
      This MR fixes our license to incorporate the new year.
      
      ## 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
      `./orbit.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
      963f3045
  10. 30 Dec, 2023 2 commits
    • Mayank Mittal's avatar
      Fixes typos in the how-to docs · 4bfb02c3
      Mayank Mittal authored
      4bfb02c3
    • Mayank Mittal's avatar
      Adds resources for learning Omniverse Kit (#343) · a9a85983
      Mayank Mittal authored
      # Description
      
      This MR adds a bunch of online resources and tutorials that users can
      find useful for mastering Omniverse. This is not a complete list, but I
      found them useful when I started.
      
      ## Type of change
      
      - This change requires a documentation update
      
      ## Checklist
      
      - [x] I have run the [`pre-commit` checks](https://pre-commit.com/) with
      `./orbit.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
      a9a85983
  11. 27 Dec, 2023 1 commit
    • Pascal Roth's avatar
      Fixes `orbit_assets` copy and bugfixes in docker and singularity (#338) · c530af44
      Pascal Roth authored
      # Description
      
      This PR fixes three issues related to cluster deployment:
      
      - remove the explicit copy of `orbit_assets` directory in the
      `container.sh` file
      - Direct bind the `logs` directory in the `run_singularity.sh` file to
      make sure that the logs are copied when the run crashed
      - Create an additional file in the docker to make sure it exists when
      binded when the singularity is run
      
      ## 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
      `./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 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: 's avatarMayank Mittal <12863862+Mayankm96@users.noreply.github.com>
      c530af44
  12. 22 Dec, 2023 8 commits
    • Mayank Mittal's avatar
      Puts the AppLauncher tutorial later in the series (#341) · 62aeea19
      Mayank Mittal authored
      # Description
      
      Currently, the very first tutorial for users is the app launcher. This
      might be a bit overwhelming to start with. This MR iterates over the app
      launcher tutorial and puts it later in the part 1 of the tutorials.
      
      ## Type of change
      
      - This change requires a documentation update
      
      ## Checklist
      
      - [x] I have run the [`pre-commit` checks](https://pre-commit.com/) with
      `./orbit.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
      62aeea19
    • Mayank Mittal's avatar
    • Mayank Mittal's avatar
      Adds announcement to the README · ceb41b81
      Mayank Mittal authored
      ceb41b81
    • Mayank Mittal's avatar
      Corrects code highlighting in troubleshooting docs (#340) · c0def57e
      Mayank Mittal authored
      # Description
      
      This MR fixes the code highlighting in the troubleshooting docs. It was
      referring to Isaac Sim stuff which we don't need to do anymore.
      
      ## 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
      `./orbit.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
      c0def57e
    • David Hoeller's avatar
      Adds current environments to the docs (#302) · 6cd0a7ba
      David Hoeller authored
      # Description
      
      Adds current environments to the docs with a description, an image, and
      a link to the environment.
      
      ## 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
      `./orbit.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
      6cd0a7ba
    • AutonomousHansen's avatar
      Enables viewport when livestreaming is enabled (#336) · 3b8c0e47
      AutonomousHansen authored
      # Description
      
      Ensures that a viewport is available when `--livestream/LIVESTREAM > 0`
      in AppLauncher. See [this
      PR](https://github.com/isaac-orbit/orbit/pull/310) for more details.
      
      ## 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
      `./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
      - [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
      3b8c0e47
    • Pascal Roth's avatar
      Fixes the line numbers of the tutorials and how-to's (#337) · 8d169da4
      Pascal Roth authored
      # Description
      
      Adjust the line numbers in the tutorials and how-to's to account for
      latest changes in the code files.
      
      ## 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
      `./orbit.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
      8d169da4
    • jsmith-bdai's avatar
      Fixes installation guide script path (#335) · 8a9c362c
      jsmith-bdai authored
      After the great tutorial refactor of 2023, the installation guide wasn't
      updated. This PR updates the path there.
      
      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 #334 
      
      <!-- 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
      
      - Bug fix (non-breaking change which fixes an issue)
      
      ## Checklist
      
      - [x] I have run the [`pre-commit` checks](https://pre-commit.com/) with
      `./orbit.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
      8a9c362c
  13. 21 Dec, 2023 9 commits
    • Mayank Mittal's avatar
    • David Hoeller's avatar
      Moves the robot configs to orbit_assets extension (#305) · 7ec1536d
      David Hoeller authored
      # Description
      
      Moves the robot configs from orbit core to orbit assets.
      
      ## 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
      `./orbit.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: 's avatarMayank Mittal <mittalma@leggedrobotics.com>
      7ec1536d
    • Mayank Mittal's avatar
      Adds invalidation of physics handles when simulation is stopped (#332) · 9a2a877b
      Mayank Mittal authored
      # Description
      
      Since we now handle the PhysX views directly, we need to make sure they
      are deleted properly when the simulation is stopped. This MR adds the
      missing implementation for invalidating the views in all sensors and
      asset classes.
      
      ## 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
      `./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
      - [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
      9a2a877b
    • Mayank Mittal's avatar
      Adds a how to guide for importing assets (#326) · 333c5f2f
      Mayank Mittal authored
      # Description
      
      This MR adds the documentation on how to import an asset into USD
      format.
      
      ## 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
      `./orbit.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
      333c5f2f
    • Mayank Mittal's avatar
      Makes sure that data collection unit test works (#331) · 89691978
      Mayank Mittal authored
      # Description
      
      This MR fixes the minor issues to ensure the data collection and
      environment tests work.
      
      ## 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
      `./orbit.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
      89691978
    • Mayank Mittal's avatar
      f794d54f
    • James Smith's avatar
      93cbeaca
    • Pascal Roth's avatar
      Add dependency to log runs with `wandb` in docker/singularity setup (#293) · 16b0b992
      Pascal Roth authored
      # Description
      
      Add a dependency to resolve an opencv-python error when trying to log
      runs with weight and biases in a singularity setup on a SLURM cluster.
      
      ## 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
      `./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 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
      16b0b992
    • AutonomousHansen's avatar
      Corrects execution command in RL training tutorial docs (#311) · a319d443
      AutonomousHansen authored
      # Description
      
      Currently, the command to play a trained RL model is lacking the flag
      that will make it use the latest checkpoint, despite that behavior
      being described as expected in the surrounding text. This MR adds the flag.
      
      It also removes a statement that 'by default' the above command will
      load the latest checkpoint, because it could be potentially confusing as
      that behavior is only 'by default' when you use the flag.
      
      ## 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
      `./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 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
      a319d443
  14. 20 Dec, 2023 3 commits
    • Mayank Mittal's avatar
      Cleans up the how-to documentation (#325) · bff476a6
      Mayank Mittal authored
      # Description
      
      This MR reviews the how-tos documentation and ensures they are all
      somewhat consistent.
      
      ## Type of change
      
      - This change requires a documentation update
      
      ## Checklist
      
      - [x] I have run the [`pre-commit` checks](https://pre-commit.com/) with
      `./orbit.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
      bff476a6
    • Mayank Mittal's avatar
      Fixes the inverse kinematics example for Franka robot (#319) · cf7a65f3
      Mayank Mittal authored
      # Description
      
      This MR makes sure the IK example works properly again. It adds the IK
      action space to the lift environment. Additionally, it fixes the state
      machine for the lift environment.
      
      ## 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
      `./orbit.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
      cf7a65f3
    • Mayank Mittal's avatar
      Fixes no update of outdated sensors (#315) · 818ff410
      Mayank Mittal authored
      # Description
      
      Our previous reasoning of not updating the sensors since the 'scene' is
      not updated does not work well on resets. It was done assuming the
      update is called right after reset. However, nowhere in the code, this
      seems to be the case anymore.
      
      This MR makes sure sensors are always updated when they are reset. The
      reset method should set values to zeros or defaults when the sensor data
      is invalid.
      
      ## 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
      `./orbit.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
      818ff410