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

refactor people description messages #5

Open
minhnh opened this issue May 28, 2019 · 2 comments
Open

refactor people description messages #5

minhnh opened this issue May 28, 2019 · 2 comments
Assignees

Comments

@minhnh
Copy link
Member

minhnh commented May 28, 2019

Currently in the repository there's Person.msg and Face.msg. Person.msg do not use Face.msg.

There's also the provided people_msgs/Person from wg-perception (but ros-<version>-people-msgs can be installed independently). This message, however, do not have fields for point clouds, RGB, or key points (for pose detection).

We should discuss and decide on a unified message to represent people.

Potentially related to b-it-bots/mas_domestic_robotics#152.

@argenos
Copy link
Contributor

argenos commented May 28, 2019

I see two options:

  • Use the people_msgs/Person in our message definition together with our additional fields
  • what you proposed beforehand: Use a similar format

I'm partial to the second. @alex-mitrevski @PatrickNa thoughts?

@minhnh
Copy link
Member Author

minhnh commented May 29, 2019

Another approach for people description messages from intel ROS 2 OpenVINO project may also give some inspiration.

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

No branches or pull requests

4 participants