-
Notifications
You must be signed in to change notification settings - Fork 45
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
Object/Person/Face views #234
Object/Person/Face views #234
Conversation
… msgs are now used
Kudos, SonarCloud Quality Gate passed! 0 Bugs |
@alex-mitrevski I turned it into a draft, this should avoid triggering reminders in Slack. I think if when the PR is opened, you mark it as a draft, then reviewers don't get added (so you don't have to do it manually each time), at least according to this:
We could also update the CODEOWNERS file and repo settings for the review process (requesting manually vs adding automatic reviewers). |
@argenos The draft PR sounds good; I'll try that next time. I think the automatic review assignment can remain, it's good to have in general. |
This PR changes the use of the
Person
,Face
, andObject
messages in the repository so that theObjectView
message (added with b-it-bots/mas_perception_msgs#11) is used when storing images and/or point clouds.