Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat/add realsense gazebo plugin #14

Merged
merged 11 commits into from
Nov 21, 2024
Merged

Conversation

Jelmerdw
Copy link
Collaborator

No description provided.

YuriVanWarmerdam and others added 10 commits November 6, 2024 10:16
Signed-off-by: Yuri van Warmerdam <[email protected]>
Signed-off-by: Yuri van Warmerdam <[email protected]>
Signed-off-by: Yuri van Warmerdam <[email protected]>
Signed-off-by: Yuri van Warmerdam <[email protected]>
Signed-off-by: Yuri van Warmerdam <[email protected]>
Signed-off-by: Yuri van Warmerdam <[email protected]>
Signed-off-by: Jelmer de Wolde <[email protected]>
Signed-off-by: Jelmer de Wolde <[email protected]>
Signed-off-by: Jelmer de Wolde <[email protected]>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change filename to empty.sdf (or perhaps something like empty_sensors.sdf or the like). As discussed, this file being here could be seen as tech debt, but is fine for now, until we can open source more assets.

<always_on>true</always_on>
<update_rate>30.0</update_rate>

<topic>${namespace}/color/image_raw</topic>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As discussed, we can consider making this camera/camera/ (or ${namespace}/${namespace} or something like that if we want to confirm more to the way the "real" camera publishes topics.

@Jelmerdw Jelmerdw merged commit dcb719a into main Nov 21, 2024
3 checks passed
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.

2 participants