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: Login only in master processes #4866

Merged
merged 8 commits into from
Sep 20, 2021
Merged

Conversation

AyushExel
Copy link
Contributor

@AyushExel AyushExel commented Sep 20, 2021

This PR allows wandb to login only in master processes

πŸ› οΈ PR Summary

Made with ❀️ by Ultralytics Actions

🌟 Summary

Improved Weights & Biases (wandb) logging setup in Ultralytics YOLOv5.

πŸ“Š Key Changes

  • πŸ“Œ Introduced a new environmental variable check to determine the process rank for distributed training.
  • βœ… Modified wandb login call to only occur on the main process or single-process runs.

🎯 Purpose & Impact

  • πŸš€ Purpose: This change is intended to prevent issues during distributed training setups where multiple processes attempt to log in to wandb simultaneously.
  • πŸ’» Impact: Users conducting distributed training will experience smoother startups without redundant wandb login attempts, potentially reducing errors and resource contention. This leads to more efficient experiment tracking and logging.

@glenn-jocher glenn-jocher merged commit 9febea7 into ultralytics:master Sep 20, 2021
@glenn-jocher
Copy link
Member

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

CesarBazanAV pushed a commit to CesarBazanAV/yolov5 that referenced this pull request Sep 29, 2021
* evolve fix

* Enable login timeout

* fix pkg

* check rank

* don't relogin
BjarneKuehl pushed a commit to fhkiel-mlaip/yolov5 that referenced this pull request Aug 26, 2022
* evolve fix

* Enable login timeout

* fix pkg

* check rank

* don't relogin
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