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

Tracker ID Added #67

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

Conversation

cyhunblr
Copy link

@cyhunblr cyhunblr commented Apr 10, 2024

PR Type

  • Feature
  • Bug fix
  • Refactor
  • Documentation
  • [] Other

Overview

  • Tracker ID added in /yolo_result topic.
  • Even though I didn't want to, I had to create my own message files. [Detection2D, Detection2DArray], and track_id added put into Detection2D.

Test

  • Successed
  • I have tested this change with gazebo
  • I have tested this change with rosbag

Attention

track_ids = results[0].boxes.id
...
if track_ids is None:
            track_ids = [0]
for bbox, cls, conf, track_id in zip(bounding_box, classes, confidence_score, track_ids):
  • If the zip is encountered as None, an error is thrown, so I'm not sure if this is the correct type of solution nonetheless it does work.

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.

1 participant