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

W&B: DDP fix #5176

Merged
merged 7 commits into from
Oct 14, 2021
Merged

W&B: DDP fix #5176

merged 7 commits into from
Oct 14, 2021

Conversation

AyushExel
Copy link
Contributor

@AyushExel AyushExel commented Oct 14, 2021

Fix: #5163

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Enhanced Weights & Biases (WandB) integration with safe import check.

πŸ“Š Key Changes

  • Added a try-except block for importing the wandb library.
  • Included an assertion to ensure that the imported wandb module is the official package, not a local directory with the same name.

🎯 Purpose & Impact

  • Purpose: The changes prevent import errors by checking whether the wandb library is properly installed and is the correct version, thus mitigating potential conflicts or crashes.
  • Impact: Users who rely on WandB for logging and experiment tracking will experience a more robust setup, reducing the risk of unexpected errors due to package issues. This contributes to a smoother workflow and easier debugging for developers utilizing the yolov5 integration with WandB.

@glenn-jocher
Copy link
Member

@AyushExel thanks! Is this ready to merge?

@AyushExel
Copy link
Contributor Author

AyushExel commented Oct 14, 2021

@glenn-jocher yes

@glenn-jocher glenn-jocher merged commit 1172ac1 into ultralytics:master Oct 14, 2021
@glenn-jocher
Copy link
Member

@AyushExel PR is merged. Thank you for your contributions to YOLOv5 πŸš€ and Vision AI ⭐

BjarneKuehl pushed a commit to fhkiel-mlaip/yolov5 that referenced this pull request Aug 26, 2022
* fix dpp with wandb disabled

* PyCharm reformat

* Fix

* fix

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.

2 participants