Skip to content

Comments

update pika teleop#4

Merged
MinnaZhong merged 1 commit intoxArm-Developer:mainfrom
MinnaZhong:main
Feb 13, 2026
Merged

update pika teleop#4
MinnaZhong merged 1 commit intoxArm-Developer:mainfrom
MinnaZhong:main

Conversation

@MinnaZhong
Copy link

Title

Short, imperative summary (e.g., "fix(robots): handle None in sensor parser"). See CONTRIBUTING.md for PR conventions.

Type / Scope

  • Type: (Bug | Feature | Docs | Performance | Test | CI | Chore)
  • Scope: (optional — name of module or package affected)

Summary / Motivation

  • One-paragraph description of what changes and why.
  • Why this change is needed and any trade-offs or design notes.

Related issues

  • Fixes / Closes: # (if any)
  • Related: # (if any)

What changed

  • Short, concrete bullets of the modifications (files/behaviour).
  • Short note if this introduces breaking changes and migration steps.

How was this tested (or how to run locally)

  • Tests added: list new tests or test files.
  • Manual checks / dataset runs performed.
  • Instructions for the reviewer

Example:

  • Ran the relevant tests:

    pytest -q tests/ -k <keyword>
  • Reproduce with a quick example or CLI (if applicable):

    lerobot-train --some.option=true

Checklist (required before merge)

  • Linting/formatting run (pre-commit run -a)
  • All tests pass locally (pytest)
  • Documentation updated
  • CI is green

Reviewer notes

  • Anything the reviewer should focus on (performance, edge-cases, specific files) or general notes.
  • Anyone in the community is free to review the PR.

@MinnaZhong MinnaZhong merged commit 8b51e30 into xArm-Developer:main Feb 13, 2026
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant