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

Feature/25 mvp integration #32

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Conversation

vangeliq
Copy link

@vangeliq vangeliq commented Oct 19, 2024

Main changes:

Picture Node:
Rewrote so this node attempts to open a folder and read the image files in the folder. This node now publishes the images as a ros2 image message called /input_image

Inference Node
Created an inference node that takes in the /input_image topic, and publishes the output image and bounding box of a yolo model. This node calls on a util class to do the preprocessing, inference and post processing.

Util Class
Added a utility class that uses the ultralytics API to do preprocessing, inference and post processing

Other changes

  • updated README to include topic subscription/publishing behaviour of nodes. Added example arguments for the new nodes
  • updated gitignore files to include .pt and some other folders
  • edit setup file to accommodate including the util classes in the ROS2 nodes.

@vangeliq vangeliq linked an issue Oct 19, 2024 that may be closed by this pull request
18 tasks
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.

MVP Integration
1 participant