Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

carmendrpr
Copy link
Contributor

@carmendrpr carmendrpr commented Aug 5, 2025

Basic Info

Info Please fill out this column
Issue(s) this addresses #799
ROS2 version tested on Humble
Aerial platform tested on Gazebo

Description of contribution in a few bullet points

  • The f330.sdf.jinja file was modified to load the gripper model with the namespace in order to avoid fixed names.

@carmendrpr carmendrpr requested review from RPS98 and pariaspe August 5, 2025 11:33
@carmendrpr carmendrpr self-assigned this Aug 5, 2025
<model name="two_fingers_gripper">
<pose
relative_to="base_link">
{{ sensor.pose }}
Copy link
Member

Choose a reason for hiding this comment

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

Pure SDF files should not include templating

Copy link
Member

Choose a reason for hiding this comment

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

Hi @pariaspe, why do we keep the SDF file in sensors if we use the jinja file?

Copy link
Member

Choose a reason for hiding this comment

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

Just in case you want to use the model without templating and our custom launching system

</geometry>
<material>
<script>
<name>"loquesea"</name>
Copy link
Member

Choose a reason for hiding this comment

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

Notice this lines

@RPS98
Copy link
Member

RPS98 commented Sep 9, 2025

Hi @carmendrpr, has you done the requested changes?

@carmendrpr
Copy link
Contributor Author

Hi @carmendrpr, has you done the requested changes?

Yes. The SDF file does not support templating. I set the default namespace to "/drone0." This namespace is commonly used as the default in most projects.

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.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.