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

Optimize descritpion to be usable also remotely #202

Open
destogl opened this issue Apr 18, 2024 · 1 comment · May be fixed by #200
Open

Optimize descritpion to be usable also remotely #202

destogl opened this issue Apr 18, 2024 · 1 comment · May be fixed by #200
Assignees

Comments

@destogl
Copy link
Member

destogl commented Apr 18, 2024

For the remote access fo the robot description, e.g., when running software on the robot and showing data on a remote computer, we need to use package://-form of accessing meshes to have paths resolved properly. Therefore we need to all this snippet to our templates:

<xacro:if value="${simulation}">
      <xacro:property
        name="file_prefix"
        value="file://$(find maha_description)"
        />
    </xacro:if>
    <xacro:unless value="${simulation}">
      <xacro:property
        name="file_prefix"
        value="package://maha_description"
      />
    </xacro:unless>

and then use <mesh filename="${file_prefix}/meshes/ everywhere.

@Nibanovic Nibanovic self-assigned this Apr 22, 2024
@Nibanovic Nibanovic linked a pull request Apr 22, 2024 that will close this issue
2 tasks
@Nibanovic
Copy link
Contributor

Nibanovic commented Apr 22, 2024

I've addressed this in a commit in gazebo_template PR, as it already has gazebo set up for me to test this.

Test:

Added the flag, generated the description package and added a dummy base_link.stl into the generated package.

Expectations:

Launching both regular and gazebo bringup should spawn with base_link.stl as geometry, without any errors from Gazebo

Result:

Everything works as expected

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 a pull request may close this issue.

2 participants