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

Object detection 2d yolov5 #360

Merged
merged 33 commits into from Nov 30, 2022
Merged

Object detection 2d yolov5 #360

merged 33 commits into from Nov 30, 2022

Conversation

vivinousi
Copy link
Collaborator

This PR introduces YOLOv5 as an inference-only addition to the toolkit

@tsampazk tsampazk self-requested a review November 28, 2022 16:27
@ad-daniel ad-daniel added test sources Run style checks test tools Test the toolkit methods labels Nov 28, 2022
@vivinousi vivinousi added the enhancement New feature or request label Nov 28, 2022
Copy link
Collaborator

@passalis passalis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you! This seems overall fine for me as an inference-only tool. I have only two minor comments.

docs/reference/object-detection-2d-yolov5.md Outdated Show resolved Hide resolved
@vivinousi vivinousi marked this pull request as draft November 29, 2022 13:36
@tsampazk tsampazk mentioned this pull request Nov 29, 2022
24 tasks
@vivinousi vivinousi marked this pull request as ready for review November 29, 2022 17:13
Copy link
Collaborator

@ad-daniel ad-daniel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very good thank you.
The documentation page should also be referenced in the index.md file, a changelog entry should be added, and a handful of details

docs/reference/object-detection-2d-yolov5.md Outdated Show resolved Hide resolved
docs/reference/object-detection-2d-yolov5.md Outdated Show resolved Hide resolved
docs/reference/object-detection-2d-yolov5.md Outdated Show resolved Hide resolved
docs/reference/object-detection-2d-yolov5.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@tsampazk tsampazk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @vivinousi, the new tool looks good. I have only some minor comments.

vivinousi and others added 9 commits November 30, 2022 12:30
Co-authored-by: ad-daniel <44834743+ad-daniel@users.noreply.github.com>
Co-authored-by: ad-daniel <44834743+ad-daniel@users.noreply.github.com>
Co-authored-by: ad-daniel <44834743+ad-daniel@users.noreply.github.com>
Co-authored-by: ad-daniel <44834743+ad-daniel@users.noreply.github.com>
Co-authored-by: Kostas Tsampazis <27914645+tsampazk@users.noreply.github.com>
Co-authored-by: Kostas Tsampazis <27914645+tsampazk@users.noreply.github.com>
…e_tutorial.ipynb

Co-authored-by: Kostas Tsampazis <27914645+tsampazk@users.noreply.github.com>
…e_tutorial.ipynb

Co-authored-by: Kostas Tsampazis <27914645+tsampazk@users.noreply.github.com>
@vivinousi
Copy link
Collaborator Author

Very good thank you. The documentation page should also be referenced in the index.md file, a changelog entry should be added, and a handful of details

Thank you Daniel, I added an entry in the index.md file as well as in the changelog, although I added today's date there which might need to be updated, in ab30d52

Copy link
Collaborator

@ad-daniel ad-daniel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All good for my side, thanks!

Copy link
Collaborator

@tsampazk tsampazk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@vivinousi vivinousi merged commit fad5d29 into develop Nov 30, 2022
@vivinousi vivinousi deleted the object-detection-2d-yolov5 branch November 30, 2022 12:12
@ad-daniel ad-daniel mentioned this pull request Nov 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request test sources Run style checks test tools Test the toolkit methods
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants