-
-
Notifications
You must be signed in to change notification settings - Fork 15.9k
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
Add segmentation and classification support for ClearML #10752
Add segmentation and classification support for ClearML #10752
Conversation
@thepycoder thanks for the PR! We'll take a look at this. |
@thepycoder I've tested it and everything works great, thank you! ps . Could be off topic here but would you know if I have to configure clearML or edit your logger to also log the best model under "artefacts"? I am new to ClearML so could be a stupid question :) |
Is there any chance for merging this? |
👋 Hello there! We wanted to let you know that we've decided to close this pull request due to inactivity. We appreciate the effort you put into contributing to our project, but unfortunately, not all contributions are suitable or aligned with our product roadmap. We hope you understand our decision, and please don't let it discourage you from contributing to open source projects in the future. We value all of our community members and their contributions, and we encourage you to keep exploring new projects and ways to get involved. For additional resources and information, please see the links below:
Thank you for your contributions to YOLO 🚀 and Vision AI ⭐ |
Will this pr be closed because nobody had time to look at this? |
@pktiuk thanks for your contribution! We apologize for the delay in reviewing your pull request. Our team is working hard to review and address all incoming requests. We appreciate your patience in this matter. Rest assured, your pull request will be reviewed and evaluated soon. Thanks again for your contribution to YOLOv5! |
@glenn-jocher I hope so |
@pktiuk we greatly appreciate your understanding and patience. Thank you for your valuable contribution to the YOLOv5 repository! Your PR will be reviewed soon. |
Signed-off-by: Glenn Jocher <glenn.jocher@ultralytics.com>
…10752) * Added ClearML instance segmentation and classification support * Cleaned up ClearML plot output * typos * Log results as plots instead of debug samples * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci --------- Signed-off-by: Glenn Jocher <glenn.jocher@ultralytics.com> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Glenn Jocher <glenn.jocher@ultralytics.com>
This PR adds ClearML support to
GenericLogger
which is used by the instance segmentation and classification training scripts, also solves issue #10710The feature list proposed in PR #8620 has been loosely followed, depending on what I had access to through the
GenericLogger
interface:The 2 features that are not yet included, aren't because I cannot access them via the existing logging interface. @glenn-jocher does it make sense to extend the
GenericLogger
for this?🛠️ PR Summary
Made with ❤️ by Ultralytics Actions
🌟 Summary
Enhancements to ClearML logging support in Ultralytics YOLOv5.
📊 Key Changes
log_scalars
.ClearmlLogger
initialization is now more robust, with exceptions caught for failed setup, and theproject_name
logic was updated to default to 'YOLOv5' whenopt.project
starts with 'runs/'.🎯 Purpose & Impact