Add OverlayImage and OverlayCamera displays and an OverlayPicker tool - #1830
Open
iory wants to merge 3 commits into
Open
Add OverlayImage and OverlayCamera displays and an OverlayPicker tool#1830iory wants to merge 3 commits into
iory wants to merge 3 commits into
Conversation
Contributor
|
Tick the box to add this pull request to the merge queue (same as
|
Displays a sensor_msgs/msg/Image as a 2D overlay on top of the 3D scene, with alpha, aspect ratio and placement properties. The shared Ogre overlay helper derives from jsk_rviz_plugins (BSD-3-Clause); decoding reuses ROSImageTexture. Signed-off-by: Iori Yanokura <ab.ioryz@gmail.com>
Dragging with the tool active moves the overlay under the cursor. The move is written through the left/top properties, so it shows in the property tree and is saved to the config. Ported from jsk_rviz_plugins (BSD-3-Clause). Signed-off-by: Iori Yanokura <ab.ioryz@gmail.com>
Composites the camera view into the main 3D view. Renders into an offscreen Ogre render texture, since rviz2's QWindow-based render window draws nothing while hidden, with overlays disabled on that viewport so the overlay texture is not fed back into itself. Signed-off-by: Iori Yanokura <ab.ioryz@gmail.com>
iory
force-pushed
the
feature/jsk-overlay
branch
from
July 31, 2026 07:08
cb0ac37 to
80797d2
Compare
Member
|
Tested on ROS 2 Rolling inside Docker running Ubuntu 26.04 LTS with a physical USB camera (
No issues observed. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Ports OverlayImage, OverlayCamera and OverlayPicker from
jsk_rviz_plugins. The camera image isdrawn as a screen-fixed 2D overlay on top of the 3D scene (OverlayImage) or composited with it
(OverlayCamera), and the tool drags those overlays around, writing the position back to the
display's Left/Top properties. Additive only: no existing display changes behaviour, and no new
dependencies.
Verified on a real USB camera (640x480 rgb8, 31 fps) — demo video below, plus unit tests for the
overlay geometry and for the picker's grid snapping. The three commits each build and test on
their own, so the OverlayCamera commit can be dropped if it is not wanted.
Generative AI: codex (gpt-5.3-codex).
jsk_overlay_demo.mp4