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

Comet changes #9328

Merged
merged 2 commits into from
Sep 8, 2022
Merged

Comet changes #9328

merged 2 commits into from
Sep 8, 2022

Conversation

DN6
Copy link
Contributor

@DN6 DN6 commented Sep 8, 2022

This PR:

  1. Adds a link to the Comet Tutorial from the main README
  2. Fixes a bug in the bbox_interval calculation resulting in division by zero error in the Comet Logger when logging predictions.

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

Enhanced logging intervals for Comet.ml in Ultralytics YOLOv5.

📊 Key Changes

  • Adjusted the logging frequency of predictions to Comet.ml when using YOLOv5.

🎯 Purpose & Impact

  • 🛠 Fixes an issue with setting the interval for logging predictions to Comet.ml to ensure more consistent behavior.
  • 🚀 Users with shorter training runs (less than 10 epochs) will now get prediction logs every epoch, enhancing the monitoring and evaluation experience.
  • 📈 This leads to better insight and tracking for small-scale experiments, potentially improving model development and debugging processes.

@DN6
Copy link
Contributor Author

DN6 commented Sep 8, 2022

@glenn-jocher Added the link to the Comet tutorial to the main README. Also found a bug this morning regarding bbox_interval and division by zero errors in the Comet Logger. So I made a fix for that as well.

@DN6
Copy link
Contributor Author

DN6 commented Sep 8, 2022

Hi @glenn-jocher. Just following up here, if we could get this merged in, we're ready to start sharing this integration with users on our side. Thanks!

@glenn-jocher glenn-jocher merged commit 5f075ee into ultralytics:master Sep 8, 2022
@glenn-jocher
Copy link
Member

@DN6 thanks for the updates. Merged!

@DN6
Copy link
Contributor Author

DN6 commented Sep 8, 2022

Thank you!

ctjanuhowski pushed a commit to ctjanuhowski/yolov5 that referenced this pull request Sep 8, 2022
* add link to comte tutorial from  main README

* fix prediction interval bug
@glenn-jocher
Copy link
Member

@DN6 you're welcome! Thank you for your contribution, and let us know if you need anything else.

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