Skip to content

waveshare WAVER physical robot - #36

Closed
stephen-derosa wants to merge 3 commits into
mainfrom
sderosa/tmp/integration-with-msg-type-field
Closed

stephen-derosa wants to merge 3 commits into
mainfrom
sderosa/tmp/integration-with-msg-type-field

Conversation

@stephen-derosa

Copy link
Copy Markdown
Collaborator

Isolated single commit re-adding the temporary inbound msg_type stopgap on top of the clean integration branch, for continued integration testing. LiveKit DataTracks do not yet carry ROS message-type metadata, so a pure consumer cannot infer an inbound track's type; this lets the bridge create the republishing publisher from a configured type.

Drop this commit (it is the exact inverse of "Drop temporary inbound msg_type stopgap (BOT-301)" on sderosa/integration-clean) once DataTracks propagate the ROS type with the track.

@linear-code

linear-code Bot commented Jul 9, 2026 •

Copy link
Copy Markdown
BOT-301 Services <> RPC

  • Align ROS2 QOS with livekit QOS (needs further work)
  • forward ROS2 services to LiveKit RPC calls
  • for LiveKit RPC calls to ROS2 services

NOTE: RPC calls currently only take in strings. So this is blocked by MCAP header in DataTracks work. cc jacobgelman to reference ticket number.

NOTE: while this can be targeted for V1.25/1.5 this work is imperitive to a complete project. If the rest of the project is complete and this is all thats left we can simply use strings and encode the ros2 messages as base64.

Review in Linear

@stephen-derosa
stephen-derosa marked this pull request as draft July 9, 2026 19:01
@stephen-derosa
stephen-derosa force-pushed the sderosa/tmp/integration-with-msg-type-field branch 2 times, most recently from 2d6c0e0 to fdd458c Compare July 13, 2026 15:30
Base automatically changed from sderosa/integration-clean to main July 13, 2026 18:00
@stephen-derosa
stephen-derosa force-pushed the sderosa/tmp/integration-with-msg-type-field branch 6 times, most recently from ddd98a5 to 07a351a Compare July 15, 2026 18:43
@stephen-derosa
stephen-derosa force-pushed the sderosa/tmp/integration-with-msg-type-field branch 3 times, most recently from ed04961 to e2a24a2 Compare July 22, 2026 21:41
waveshare_driver: get imu, send control commands

test_utilities: test robot interfacing without ROS

nav2 integration: params and composition

cyclone dds middleware
@stephen-derosa
stephen-derosa force-pushed the sderosa/tmp/integration-with-msg-type-field branch from d929205 to 27b3a11 Compare July 22, 2026 22:32
@stephen-derosa stephen-derosa changed the title TEMPORARY: require explicit msg_type for inbound topics (BOT-301) waveshare WEVER physical robot Jul 24, 2026
@stephen-derosa stephen-derosa changed the title waveshare WEVER physical robot waveshare WAVER physical robot Jul 24, 2026
@stephen-derosa

Copy link
Copy Markdown
Collaborator Author

subsumed by #58

@stephen-derosa
stephen-derosa deleted the sderosa/tmp/integration-with-msg-type-field branch July 27, 2026 18:35
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