Skip to content

W&B: DDP fix#5176

Merged
glenn-jocher merged 7 commits intoultralytics:masterfrom
AyushExel:ddp_fix
Oct 14, 2021
Merged

W&B: DDP fix#5176
glenn-jocher merged 7 commits intoultralytics:masterfrom
AyushExel:ddp_fix

Conversation

@AyushExel
Copy link
Copy Markdown
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
Copy Markdown
Member

@AyushExel thanks! Is this ready to merge?

@AyushExel
Copy link
Copy Markdown
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
Copy Markdown
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 <[email protected]>
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