Skip to content

[1.7] Backport Sharpa Avatar health and lifecycle fixes (#1347) - #1480

Open
fvaloppi-nv wants to merge 1 commit into
release/1.7.xfrom
codex/backport-pr-1347-1.7
Open

fvaloppi-nv wants to merge 1 commit into
release/1.7.xfrom
codex/backport-pr-1347-1.7

Conversation

@fvaloppi-nv

Copy link
Copy Markdown
Contributor

Backport #1347 to release/1.7.x so Sharpa Avatar reports independent dongle, glove, and dataset health and recovers after hardware reconnects. Includes dynamic XDev rebinding, stale haptic command expiry, and retry handling that prevents cached commands from replaying after reconnect.

Exact, conflict-free cherry-pick of 834ecad34e2224b5d1add21e1123343b51e737ee; the stable patch ID matches the original merged PR. The Avatar SDK production pin remains 1.7.3.

Validation:

  • SKIP=check-copyright-year pre-commit run --all-files passed.
  • clang-format 14 verification passed for all touched C++ files.
  • Built and ran deviceio_trackers_tests, live_trackers_tests, and tracker_metadata_tests: all 45 test cases passed.
  • make -C docs current-docs passed with warnings treated as errors.
  • Sharpa plugin compilation/tests and physical glove validation were not run because the Avatar SDK and hardware are unavailable here. Python runtime tests were not run; this validation build disabled Python bindings.

Signed-off-by: zihan <zihan.jiang@sharpa.com>
(cherry picked from commit 834ecad)
Signed-off-by: fvaloppi-nv <fvaloppi@nvidia.com>
@coderabbitai

coderabbitai Bot commented Nov 11, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

Run configuration

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

@github-actions

Copy link
Copy Markdown
Contributor

✅ Preview deployed: https://NVIDIA.github.io/IsaacCapture/preview/pr-1480/

Built from commit 502c42e. It may take a minute for the updated preview to appear.

This branch was successfully deployed

1 active deployment
dev — 502c42e4 Deployed Nov 11, 2026 by fvaloppi-nv via publish-wheel #6773
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants