Skip to content

Optimizes Kit XR Teleop CPU time#3487

Merged
kellyguo11 merged 1 commit into
isaac-sim:release/2.3.0from
hougantc-nvda:dev/hougantc/kitxr-optimization
Sep 18, 2025
Merged

Optimizes Kit XR Teleop CPU time#3487
kellyguo11 merged 1 commit into
isaac-sim:release/2.3.0from
hougantc-nvda:dev/hougantc/kitxr-optimization

Conversation

@hougantc-nvda

@hougantc-nvda hougantc-nvda commented Sep 17, 2025

Copy link
Copy Markdown
Contributor

Description

With kitxr 107.3.111, we can save cpu time by skipping writes to usd stage since all the data is in fabric. This gives us back 11ms (on my machine) on the main thread. Additionally, texture streaming was bottlenecking the render thread. Disabling gave us back 11ms on the renderthread. Together, it gave me back 11ms.

Fixes # (issue)

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Screenshots

Please attach before and after screenshots of the change if applicable.

Checklist

  • I have read and understood the contribution guidelines
  • I have run the pre-commit checks 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

With kitxr 107.3.111, we can save cpu time by skipping writes to usd stage since all the data is in fabric. On the tested workstation, it saved 11ms on the main thread. Additionally, texture streaming was bottlenecking the render thread. Disabling resulted in 11ms savings on the renderthread. Together, it resulted in 11ms on the tested workstation as those two bottlenecked each other.
@github-actions github-actions Bot added documentation Improvements or additions to documentation asset New asset feature or request isaac-sim Related to Isaac Sim team isaac-mimic Related to Isaac Mimic team infrastructure labels Sep 17, 2025
@hougantc-nvda hougantc-nvda changed the base branch from main to release/2.3.0 September 17, 2025 20:03
@hougantc-nvda hougantc-nvda changed the title Dev/hougantc/kitxr optimization Optimize Kit XR Teleop CPU time Sep 17, 2025
@kellyguo11 kellyguo11 changed the title Optimize Kit XR Teleop CPU time Optimizes Kit XR Teleop CPU time Sep 17, 2025
@kellyguo11 kellyguo11 merged commit 91eac99 into isaac-sim:release/2.3.0 Sep 18, 2025
1 of 2 checks passed
hougantc-nvda added a commit to hougantc-nvda/IsaacLab that referenced this pull request Nov 10, 2025
# Description
With kitxr 107.3.111, we can save cpu time by skipping writes to usd
stage since all the data is in fabric. This gives us back 11ms (on my
machine) on the main thread. Additionally, texture streaming was
bottlenecking the render thread. Disabling gave us back 11ms on the
renderthread. Together, it gave me back 11ms.

<!--
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

💡 Please try to keep PRs small and focused. Large PRs are harder to
review and merge.
-->

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 read and understood the [contribution
guidelines](https://isaac-sim.github.io/IsaacLab/main/source/refs/contributing.html)
- [x] I have run the [`pre-commit` checks](https://pre-commit.com/) with
`./isaaclab.sh --format`
- [ ] I have made corresponding changes to the documentation
- [x] My changes generate no new warnings
- [ ] I have added tests that prove my fix is effective or that my
feature works
- [ ] I have updated the changelog and the corresponding version in the
extension's `config/extension.toml` file
- [ ] 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
-->
hougantc-nvda added a commit to hougantc-nvda/IsaacLab that referenced this pull request Nov 10, 2025
# Description
With kitxr 107.3.111, we can save cpu time by skipping writes to usd
stage since all the data is in fabric. This gives us back 11ms (on my
machine) on the main thread. Additionally, texture streaming was
bottlenecking the render thread. Disabling gave us back 11ms on the
renderthread. Together, it gave me back 11ms.

<!--
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

💡 Please try to keep PRs small and focused. Large PRs are harder to
review and merge.
-->

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 read and understood the [contribution
guidelines](https://isaac-sim.github.io/IsaacLab/main/source/refs/contributing.html)
- [x] I have run the [`pre-commit` checks](https://pre-commit.com/) with
`./isaaclab.sh --format`
- [ ] I have made corresponding changes to the documentation
- [x] My changes generate no new warnings
- [ ] I have added tests that prove my fix is effective or that my
feature works
- [ ] I have updated the changelog and the corresponding version in the
extension's `config/extension.toml` file
- [ ] 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
-->
@hougantc-nvda hougantc-nvda deleted the dev/hougantc/kitxr-optimization branch February 19, 2026 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

asset New asset feature or request documentation Improvements or additions to documentation infrastructure isaac-mimic Related to Isaac Mimic team isaac-sim Related to Isaac Sim team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants