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

Lucas S. files for the April tags environment #107

Open
wants to merge 10 commits into
base: kinetic-devel
Choose a base branch
from
Open

Lucas S. files for the April tags environment #107

wants to merge 10 commits into from

Conversation

LucasJSch
Copy link

@LucasJSch LucasJSch commented Oct 15, 2019

Added files for creating an environment for testing the April tags package of ROS.

Environment

tag_simulation

Tags detected by the apriltag_ros package

detected_tags

Frames detected by the apriltag_ros package

tag_detection

@LucasJSch LucasJSch changed the title Lucas S. files for the April tags environment [WIP] Lucas S. files for the April tags environment Oct 15, 2019
@LucasJSch
Copy link
Author

Ready for review

@eborghi10
Copy link
Member

To do:

  • The april tag of the robot has yet to be correctly spawned. The geometric figure is spawning correctly, but not the texture.

Was this already addressed? Also is this PR "[WIP]"?

@LucasJSch
Copy link
Author

LucasJSch commented Oct 16, 2019

To do:

  • The april tag of the robot has yet to be correctly spawned. The geometric figure is spawning correctly, but not the texture.

Was this already addressed? Also is this PR "[WIP]"?

Yes, it has. Forgot to delete it. Also forgot to delete the "[WIP]".

@LucasJSch LucasJSch changed the title [WIP] Lucas S. files for the April tags environment Lucas S. files for the April tags environment Oct 16, 2019
Copy link
Member

@eborghi10 eborghi10 left a comment

Choose a reason for hiding this comment

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

I made some comments, PTAL.

ca_description/urdf/april_tag.xacro Outdated Show resolved Hide resolved
ca_description/urdf/april_tag.xacro Outdated Show resolved Hide resolved
ca_gazebo/launch/create_april_tags.launch Outdated Show resolved Hide resolved
ca_gazebo/launch/create_april_tags.launch Outdated Show resolved Hide resolved
ca_gazebo/models/create_april_tags/model.sdf Outdated Show resolved Hide resolved
ca_gazebo/models/create_camera/model.sdf Outdated Show resolved Hide resolved
ca_gazebo/worlds/april_tags.world Outdated Show resolved Hide resolved
ca_gazebo/worlds/april_tags.world Outdated Show resolved Hide resolved
ca_gazebo/worlds/camera_tutorial.world Outdated Show resolved Hide resolved
ca_gazebo/worlds/wall_object.xacro Outdated Show resolved Hide resolved
ca_gazebo/launch/create_april_tags.launch Outdated Show resolved Hide resolved
ca_gazebo/launch/create_april_tags.launch Outdated Show resolved Hide resolved
ca_gazebo/models/create_camera/model.sdf Outdated Show resolved Hide resolved
ca_gazebo/models/create_camera/model.sdf Outdated Show resolved Hide resolved
ca_gazebo/models/create_camera/model.sdf Outdated Show resolved Hide resolved
ca_gazebo/models/create_camera/model.sdf Outdated Show resolved Hide resolved
ca_gazebo/models/create_camera/model.sdf Show resolved Hide resolved
ca_gazebo/worlds/empty.world Outdated Show resolved Hide resolved
ca_gazebo/worlds/empty.world Outdated Show resolved Hide resolved
navigation/ca_move_base/maps/empty.yaml Outdated Show resolved Hide resolved
ca_description/urdf/april_tag.xacro Outdated Show resolved Hide resolved
To do: decrease the size of the environment for using smaller image files from camera.
To do: upload apriltag_ros as as internal package in create_autonomy.
Copy link
Collaborator

@serraramiro1 serraramiro1 left a comment

Choose a reason for hiding this comment

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

Left some comments, PTAL

ca_gazebo/launch/create_april_tags.launch Outdated Show resolved Hide resolved
ca_gazebo/models/create_camera/model.sdf Outdated Show resolved Hide resolved
ca_gazebo/models/create_camera/model.sdf Show resolved Hide resolved
ca_gazebo/models/create_camera/model.sdf Outdated Show resolved Hide resolved
@serraramiro1
Copy link
Collaborator

serraramiro1 commented Oct 21, 2019

Also, fix the merge conflicts please.

<xacro:macro name="april_tag" params="name_prefix base_link *origin">

<link name="${name_prefix}_april_tag_link">
<visual>
Copy link
Member

Choose a reason for hiding this comment

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

I just realized about this. Is the visual tag mandatory? Since you only want to apply a texture, do you need to create this visual block?

Copy link
Author

Choose a reason for hiding this comment

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

If I delete this, the Apriltag doesn't appear. This xacro corresponds to the Apriltag of the robot.
How can a texture be applied if there is no visual object to apply it to?

ca_gazebo/launch/create_april_tags.launch Outdated Show resolved Hide resolved
ca_gazebo/launch/create_april_tags.launch Outdated Show resolved Hide resolved
ca_gazebo/models/create_april_tags/model.sdf Show resolved Hide resolved
ca_gazebo/models/create_house/model.sdf Outdated Show resolved Hide resolved
ca_gazebo/package.xml Outdated Show resolved Hide resolved
@@ -2,5 +2,5 @@ free_thresh: 0.196
image: empty.pgm
negate: 0
occupied_thresh: 0.65
origin: [-6.8999999999999915, -5.8999999999999915, 0.0]
origin: [0.0, 0.0, 0.0]
Copy link
Member

Choose a reason for hiding this comment

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

Rebase against master.

@eborghi10 eborghi10 force-pushed the kinetic-devel branch 3 times, most recently from 23e75d6 to 05fa7b9 Compare January 11, 2020 23:40
@eborghi10 eborghi10 linked an issue Jun 12, 2020 that may be closed by this pull request
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.

Ground truth with AprilTags
4 participants