Skip to content

Add OverlayParameters.multibeam_enabled - #298

Draft
jp-pino wants to merge 1 commit into
masterfrom
jp-pino/multibeam-overlay-enable
Draft

Add OverlayParameters.multibeam_enabled#298
jp-pino wants to merge 1 commit into
masterfrom
jp-pino/multibeam-overlay-enable

Conversation

@jp-pino

@jp-pino jp-pino commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Adds bool multibeam_enabled = 25 to OverlayParameters so the app can toggle whether the multibeam sonar fan overlay is drawn onto recordings. Split out of #278 so the recorded-fan overlay can land separately from the color filter / RTSP work.

Linked PRs (multibeam overlay on recordings)

  1. BluEye-Robotics/ProtocolDefinitions — this PR
  2. BluEye-Robotics/p2_msgs#54 — OverlayParameters.msg mirror
  3. BluEye-Robotics/p2_drone#1066 — mapping_overlay_parameters.cpp
  4. BluEye-Robotics/gst_rtsp_record#297 — mbezoverlay in the record pipeline, gated by this flag

Flow: app → OverlayParameters.multibeam_enablednode_commOverlayNode republishes on /overlay/get_parameters (tyndall) → UltraRecordApp reads it at record start and inserts mbezoverlay. Default false: nothing changes for apps that never set it.

🤖 Generated with Claude Code

https://claude.ai/code/session_01VPvg8dPeNaaExzg3iCnzCF

Lets the app toggle whether the multibeam sonar fan overlay is drawn onto
recordings.
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