Skip to content

Convert id to string#22

Merged
Kukanani merged 3 commits intoros-perception:kinetic-develfrom
mistermult:kinetic-devel
Jul 19, 2019
Merged

Convert id to string#22
Kukanani merged 3 commits intoros-perception:kinetic-develfrom
mistermult:kinetic-devel

Conversation

@mistermult
Copy link
Copy Markdown
Contributor

@mistermult mistermult commented Jul 15, 2019

  • Add a field object_id of type string, e.g. to track one object across multiple frames (tracking).
  • Modify type of (class) id to type string.

See my comment #18 (comment) for the rationale of the design.

@Kukanani Kukanani changed the title Add object_id of type string. Convert id to string Jul 19, 2019
@Kukanani
Copy link
Copy Markdown
Collaborator

Updated to only change the type of the id field based on discussion in #18.

Thanks for the PR!

@Kukanani Kukanani merged commit 866d02f into ros-perception:kinetic-devel Jul 19, 2019
Kukanani added a commit that referenced this pull request Oct 4, 2019
Kukanani added a commit that referenced this pull request Oct 4, 2019
Kukanani added a commit that referenced this pull request Apr 16, 2021
SteveMacenski pushed a commit that referenced this pull request Apr 16, 2021
* Revert "add tracking ID to the Detection Message (#19)"

This reverts commit 0d20a43.

* Revert "Convert id to string (#22)"

This reverts commit 866d02f.
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