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

World and Apriltags #108

Open
wants to merge 1 commit into
base: kinetic-devel
Choose a base branch
from

Conversation

stevendes
Copy link

@stevendes stevendes commented Oct 15, 2019

default_camera_2_link_camera_2_link(3)-0042

World and Apriltags

Created the world with 3 cameras and 2 apriltags.

Still missing launchfile and apriltag on the create robot model.

World file with some commented code that needs to be erased.

Cameras still missing the model file, for now they are created right in the .world file.

UPDATE

tag2

tags

  • Addressed all the comments

  • Include in the launch file the apriltag node run and the tf_state_publisher

@stevendes
Copy link
Author

stevendes commented Oct 16, 2019

default_Camera_link_camera_2_link(1)-0176

UPDATE 1

Added launch file.

Added apriltag on Create Robot.

Erased two unnecessary cameras.

Still need to address:

Optical Frame for the camera

READY FOR REVIEW

@stevendes stevendes changed the title [WIP]World and Apriltags World and Apriltags Oct 16, 2019
@eborghi10
Copy link
Member

Still need to address:

Optical Frame for the camera

Did you add this feature? If not, please, do it.

ca_description/urdf/create_apriltag.xacro Outdated Show resolved Hide resolved
ca_description/urdf/create_apriltag.xacro Outdated Show resolved Hide resolved
ca_description/urdf/create_apriltag.xacro Outdated Show resolved Hide resolved
ca_description/urdf/create_apriltag.xacro Outdated Show resolved Hide resolved
ca_description/urdf/create_apriltag.xacro Outdated Show resolved Hide resolved
ca_gazebo/models/Camera_world/Apriltag1/model.config Outdated Show resolved Hide resolved
ca_gazebo/models/Camera_world/Apriltag2/model.sdf Outdated Show resolved Hide resolved
ca_gazebo/models/Camera_world/Camera/model.config Outdated Show resolved Hide resolved
ca_gazebo/models/Camera_world/Camera/model.sdf Outdated Show resolved Hide resolved
ca_gazebo/worlds/camera_world.world Outdated Show resolved Hide resolved
@stevendes
Copy link
Author

stevendes commented Oct 21, 2019

UPDATE 2

  • Added "Optical Frame for the camera"
  • Addressed all comments
  • Change some minor details

ca_gazebo/models/camera_world/camera/model.sdf Outdated Show resolved Hide resolved
ca_gazebo/models/camera_world/camera/model.sdf Outdated Show resolved Hide resolved
</noise>
</camera>
<plugin name="camera_controller" filename="libgazebo_ros_camera.so">
<alwaysOn>true</alwaysOn>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is alwaysOn a valid plugin tag?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Check updateRate too

Copy link
Collaborator

Choose a reason for hiding this comment

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

check here

Copy link
Author

Choose a reason for hiding this comment

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

To what I saw, always_on is a tag for the sensor and alwaysOn is used in plugins, same case with updateRate

ca_description/urdf/create_base.xacro Outdated Show resolved Hide resolved
ca_description/urdf/create_apriltag.xacro Show resolved Hide resolved
ca_gazebo/models/camera_world/camera/model.sdf Outdated Show resolved Hide resolved
ca_gazebo/models/camera_world/camera/model.sdf Outdated Show resolved Hide resolved
Added world and models

Big changes,added launch file, apriltag on create robot, erased to cameras, cleaned the world file

Changed the apriltags and camera plugin

fixup! Adding new world

Change apriltags for improve the visualization

Added in launch file apriltag and tf_state nodes
@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
3 participants