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

Refactor Loggers : Move code outside train.py #9241

Merged
merged 7 commits into from
Sep 1, 2022

Conversation

AyushExel
Copy link
Contributor

@AyushExel AyushExel commented Sep 1, 2022

@glenn-jocher most loggers will/are pushing for dataset/model artifact registry service. They already want to use custom dataset links. This interface attempts to provide a cleaner interface for that.

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

Improved dataset artifact management for remote runs in YOLOv5.

📊 Key Changes

  • Consolidated data_dict retrieval from remote sources to a single location.
  • Enabled dataset reference update during resumption of training.

🎯 Purpose & Impact

  • Enhances code maintainability by centralizing logic related to remote dataset handling.
  • 🔄 Ensures consistent dataset usage when training is resumed, potentially improving user experience and model performance.

@glenn-jocher
Copy link
Member

@AyushExel looks good, merging PR!

@glenn-jocher glenn-jocher merged commit 2d082a0 into ultralytics:master Sep 1, 2022
ctjanuhowski pushed a commit to ctjanuhowski/yolov5 that referenced this pull request Sep 8, 2022
* update

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* update

* update

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

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>
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

2 participants