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

Adding team base #411

Merged
merged 4 commits into from
May 11, 2020
Merged

Adding team base #411

merged 4 commits into from
May 11, 2020

Conversation

nkoenig
Copy link
Contributor

@nkoenig nkoenig commented May 1, 2020

This adds a teambase. The teambase cannot move, an is permanently in the staging area. It's specified in the same manner as other robots, but the name and config must be "teambase" or "team_base". Capitalization doesn't matter.

Example

ign launch cave_circuit.ign worldName:=simple_cave_01 robotName1:=teambase robotConfig1:=teambase

You should see a small black box in the staging area. It has no collision object, just a visual.

Only 1 team base is allowed.

@pauljurczak
Copy link

What is its purpose? Is it documented somewhere?

@nkoenig
Copy link
Contributor Author

nkoenig commented May 4, 2020

A wiki page will come shortly. This gives a team the ability to have a zero-cost computer, the Team Base, in the staging area. You'll be able to assign a docker image to the Team Base, just as you would a regular robot.

The Team Base cannot move, and you can only have one. This is also optional.

" </visual>\n"\
" </link>\n"\
" <!-- Publish robot state information -->\n"\
" <plugin filename=\"libignition-gazebo-pose-publisher-system.so\"\n"\
Copy link
Contributor

Choose a reason for hiding this comment

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

I think we don't need to use pose publisher system for this simple model and instead we can just use a static_transform_publisher in teambase_topics.launch, which helps to reduce data published to the /tf topic

e.g. the teambase_topics.launch would then be:

<?xml version="1.0"?>
<launch>
  <node pkg="tf2_ros" type="static_transform_publisher" name="tf_teambase_static" args="0 0 0.05 0 0 0 TeamBase TeamBase/link"/>
</launch>

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, this should be fixed in 3225249

<robot name="TeamBase" xmlns:xacro="http://ros.org/wiki/xacro">
<xacro:property name="robot_namespace" value="TeamBase"/>

<link name='${robot_namespace}/base_link'>
Copy link
Contributor

Choose a reason for hiding this comment

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

change base_link to link to match the actual link name of the TeamBase model?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed in 3225249

@nkoenig nkoenig merged commit 5ef7adb into cave_feature_release1 May 11, 2020
@nkoenig nkoenig deleted the teambase branch December 10, 2020 22:31
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