Skip to content

feat: add sensor observations - #3

Open
han-xudong wants to merge 2 commits into
robot-i-o:mainfrom
han-xudong:han-xudong/add-sensor-observations
Open

han-xudong wants to merge 2 commits into
robot-i-o:mainfrom
han-xudong:han-xudong/add-sensor-observations

Conversation

@han-xudong

@han-xudong han-xudong commented Jul 10, 2026

Copy link
Copy Markdown

Summary

  • add generic *_sensor station fields that resolve to rio_hw.sensors
  • start configured sensor clients in Env and include their state under step.observation.sensors
  • preserve sensor state through recorder / loader flattening and reconstruction
  • add rio-list-sensors and station configuration documentation
  • cover sensor factory, env observation, loader roundtrip, and listing behavior

Issue

Fixes #5.

Stacking

This PR is stacked on #2 (test: fix contributing test setup), which fixes #4. The branch includes #2 so the CONTRIBUTING command uv run --extra dev pytest -n 4 can be run on this feature branch. After #2 merges, this branch should be rebased onto main; the remaining diff is the sensor feature only.

Validation

  • uv run --extra dev pytest -n 4 -q
  • uvx ruff check docs/workflow/station_cfg.md pyproject.toml rio/_scripts/list_available.py rio/_tests/test_sensors.py rio/data/loader.py rio/envs/env.py rio/envs/factory.py rio/schema.py
  • uvx ruff format --check pyproject.toml rio/_scripts/list_available.py rio/_tests/test_sensors.py rio/data/loader.py rio/envs/env.py rio/envs/factory.py rio/schema.py
  • uvx typos --diff docs/workflow/station_cfg.md pyproject.toml rio/_scripts/list_available.py rio/_tests/test_sensors.py rio/data/loader.py rio/envs/env.py rio/envs/factory.py rio/schema.py

@netlify

netlify Bot commented Jul 10, 2026

Copy link
Copy Markdown

Deploy Preview for robotio-docs ready!

Name Link
🔨 Latest commit 4ccef35
🔍 Latest deploy log https://app.netlify.com/projects/robotio-docs/deploys/6a505c693aa49d00088d1941
😎 Deploy Preview https://deploy-preview-3--robotio-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

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.

Add generic sensor observations for rio-hw sensors Make the documented contributing test command pass

1 participant