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

Add support for more Marker geometry types #63

Merged
merged 5 commits into from
Oct 15, 2020
Merged

Conversation

Sarath18
Copy link
Collaborator

Part 2 of #60

Add support for the following geometry types:

  • Sphere List
  • Cube List
  • Text markers
  • Mesh markers

Signed-off-by: Sarathkrishnan Ramesh <sarathkrishnan99@gmail.com>
Signed-off-by: Sarathkrishnan Ramesh <sarathkrishnan99@gmail.com>
Signed-off-by: Sarathkrishnan Ramesh <sarathkrishnan99@gmail.com>
* Add method for creating arrow marker

* Add method for converting pose message

Signed-off-by: Sarathkrishnan Ramesh <sarathkrishnan99@gmail.com>
Copy link
Collaborator

@ahcorde ahcorde left a comment

Choose a reason for hiding this comment

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

LGTM

* Add missing visualization_msgs dependency

* Load marker display plugin by topic

* Fix mesh file path extraction

Signed-off-by: Sarathkrishnan Ramesh <sarathkrishnan99@gmail.com>
@Sarath18 Sarath18 requested a review from ahcorde October 15, 2020 08:26
@Sarath18
Copy link
Collaborator Author

Made some minor improvements can you please review the last commit?

@Sarath18 Sarath18 merged commit 6c95fe7 into master Oct 15, 2020
@Sarath18 Sarath18 deleted the marker_display_2 branch October 15, 2020 08: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.

2 participants