Skip to content

Work: Complete per-step observation noise coverage for Sharpa hand in-hand envs #306

Description

@TATP-233

Background

Issue #125 mixed multiple robots into one work item. This issue now tracks the Sharpa hand slice only. Allegro is no longer in scope for this tracking set.

Current Repo Audit

  • SharpaInhandRotation already injects joint observation noise in src/unilab/envs/manipulation/sharpa_inhand/rotation.py.
  • SharpaInhandRotationGrasp inherits from SharpaInhandRotationEnv, so the shared observation path should be covered as one robot slice.
  • I did not find dedicated Sharpa obs-noise tests in tests/.

Remaining Work

  • Confirm both Sharpa in-hand env variants apply per-step observation noise through the shared observation path.
  • Add dedicated unit tests for positive-noise and zero-noise cases.
  • Record the validation command/result in the PR.

Definition of Done

Scope

  • Robot: Sharpa hand
  • Includes: SharpaInhandRotation, SharpaInhandRotationGrasp
  • Excludes: G1 / Go1 / Go2 / Allegro

Activity

  1. changed the title [-]Work: Complete per-step observation noise coverage for Allegro in-hand rotation envs[/-] [+]Work: Complete per-step observation noise coverage for Sharpa hand in-hand envs[/+] on Apr 18, 2026
  2. TATP-233 commented on May 4, 2026

    @TATP-233
    CollaboratorAuthor
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

enhancementNew feature or request

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions