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

Detection support #60

Merged
merged 17 commits into from
Dec 3, 2022
Merged

Detection support #60

merged 17 commits into from
Dec 3, 2022

Conversation

AyushExel
Copy link
Contributor

@AyushExel AyushExel commented Dec 1, 2022

TODO:

  • detection plotting

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Improved GitHub workflow for YOLOv5 model training and validation, and updates to loss and plotting functions. πŸš€

πŸ“Š Key Changes

  • Added detect task to the CI workflow with training and validation steps.
  • Rearranged positions of ap (Average Precision) and f1 in performance metric dictionaries.
  • Updated the plotting functions to include output features like image saving and improved image grid representation.
  • Introduced a detection training and validation module under ultralytics/yolo/v8/detect.
  • Plotting function changes support new functionality like grid plots with image labels.
  • Created new API endpoints for detection tasks (detect).

🎯 Purpose & Impact

  • πŸ‹οΈ Ensures automated testing of detection tasks (training and validation) through CI, enhancing software quality.
  • πŸ“Š Streamlines performance metric presentations, promoting consistency and clarity within results.
  • 🎨 Improves visualization of results, enabling better interpretation of data during model development processes.
  • πŸ“‘ New detection modules extend the capabilities of the library, allowing users to easily train and validate detection models.
  • πŸ‘©β€πŸ’» Developers and users gain from more robust and feature-complete tools for object detection with YOLO.

@AyushExel
Copy link
Contributor Author

@Laughing-q took a first pass on detection. I inherited the segmentationTrainer and things seem to work without errors. But not sure about the correctness.

@Laughing-q
Copy link
Member

@AyushExel Got some time testing this pr. If I found something to update, can I directly commit in this pr?

@AyushExel
Copy link
Contributor Author

@Laughing-q yes please go ahead

@Laughing-q
Copy link
Member

@AyushExel okay the results of detect training seem to be correct (both single-gpu and ddp). Btw I added the plot.

@AyushExel
Copy link
Contributor Author

@Laughing-q ohh awesome. I'm merging this then.. Now we have all 3 tasks yay!!!

@AyushExel AyushExel merged commit 7ec7cf3 into main Dec 3, 2022
@AyushExel AyushExel deleted the detect branch December 3, 2022 15:31
@glenn-jocher
Copy link
Member

@AyushExel @Laughing-q detection training and val is working now??

Wow ok then I can start porting over TAL/DFL loss updates from yolov5:exp13 branch?

@AyushExel
Copy link
Contributor Author

@glenn-jocher yes both detection and segmentation training now reproduce yolov5 both in single-gpu and ddp mode.
You can start porting stuff over.
There is one small problem to be fixed.. increment_path creates multiple paths in DDP mode. It's not a complicated problem, we'll fix it next

@glenn-jocher
Copy link
Member

@AyushExel ok got it!

0iui0 pushed a commit to 0iui0/ultralytics that referenced this pull request Jan 3, 2024
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Laughing-q <1185102784@qq.com>
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.

None yet

3 participants