Skip to content

feat(cockpit): Video Panel - #3263

Open
paul-nechifor wants to merge 3 commits into
mainfrom
paul/feat/t5
Open

feat(cockpit): Video Panel#3263
paul-nechifor wants to merge 3 commits into
mainfrom
paul/feat/t5

Conversation

@paul-nechifor

@paul-nechifor paul-nechifor commented Jul 29, 2026

Copy link
Copy Markdown
Contributor
  • Added the VideoPanel which shows the color_image published by the bridge (jpeg.v1 encoded). It also displays FPS and staleness.
  • Fixed a but related to the Mujoco process. Its output pipe was never drained and so it would periodically stop publishing data.
  • Only encode and send when there's a listener.

@codecov

codecov Bot commented Jul 29, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 87.24832% with 38 lines in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
dimos/robot/unitree/mujoco_connection.py 48.38% 9 Missing and 7 partials ⚠️
dimos/e2e_tests/test_cockpit_browser.py 0.00% 11 Missing ⚠️
dimos/robot/unitree/test_mujoco_connection.py 94.44% 5 Missing and 1 partial ⚠️
dimos/protocol/pubsub/impl/jpeg_shm.py 33.33% 2 Missing ⚠️
dimos/web/relay_bridge/test_wt_client.py 94.87% 0 Missing and 2 partials ⚠️
dimos/agents/skills/person_follow.py 50.00% 1 Missing ⚠️
@@            Coverage Diff             @@
##             main    #3263      +/-   ##
==========================================
- Coverage   75.20%   75.11%   -0.09%     
==========================================
  Files        1128     1130       +2     
  Lines      107954   108272     +318     
  Branches     9751     9771      +20     
==========================================
+ Hits        81188    81333     +145     
- Misses      23955    24174     +219     
+ Partials     2811     2765      -46     
Flag Coverage Δ
OS-ubuntu-24.04-arm 68.95% <87.24%> (+0.09%) ⬆️
OS-ubuntu-latest 71.01% <87.24%> (+0.08%) ⬆️
Py-3.10 71.01% <87.24%> (+0.08%) ⬆️
Py-3.11 71.00% <87.24%> (+0.08%) ⬆️
Py-3.12 71.00% <87.24%> (+0.08%) ⬆️
Py-3.13 71.00% <87.24%> (+0.08%) ⬆️
Py-3.14 71.01% <87.24%> (+0.08%) ⬆️
Py-3.14t 71.01% <87.24%> (+0.09%) ⬆️
SelfHosted-Large 29.07% <20.33%> (-0.02%) ⬇️
SelfHosted-macOS ?

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
dimos/msgs/sensor_msgs/Image.py 71.04% <100.00%> (+13.18%) ⬆️
dimos/utils/turbojpeg.py 100.00% <100.00%> (ø)
dimos/web/relay_bridge/_wt_session.py 89.87% <100.00%> (+3.33%) ⬆️
dimos/web/relay_bridge/manifest.py 100.00% <100.00%> (ø)
dimos/web/relay_bridge/protocol.py 100.00% <100.00%> (ø)
dimos/web/relay_bridge/relay_bridge_module.py 93.05% <100.00%> (+1.33%) ⬆️
dimos/web/relay_bridge/relay_process.py 80.07% <100.00%> (+2.12%) ⬆️
dimos/web/relay_bridge/test_protocol.py 100.00% <100.00%> (ø)
dimos/web/relay_bridge/test_relay_bridge_module.py 99.01% <100.00%> (+0.67%) ⬆️
dimos/web/relay_bridge/test_relay_e2e.py 99.11% <100.00%> (+2.31%) ⬆️
... and 8 more

... and 32 files with indirect coverage changes

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions github-actions Bot added ready-to-merge Required CI checks have passed on this PR and removed ready-to-merge Required CI checks have passed on this PR labels Jul 29, 2026
Base automatically changed from paul/feat/t3-p3 to main July 29, 2026 11:15
@paul-nechifor paul-nechifor changed the title WIP: T5 feat(cockpit): Video Panel Jul 30, 2026
@paul-nechifor
paul-nechifor enabled auto-merge July 30, 2026 02:00
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