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

Uninstall wandb from notebook environments #11730

Merged
merged 3 commits into from
Jun 18, 2023
Merged

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented Jun 18, 2023

Due to undesired behavior in https://www.kaggle.com/code/ultralytics/yolov8/comments#2306977

🤖 Generated by Copilot at e9b92e2

Summary

🐛📝🚀

Prevent wandb account creation prompt and hang in notebook_init function. Add check for wandb installation before uninstalling it in utils/__init__.py.

notebook_init waits
wandb prompts for account
check before uninstall

Walkthrough

  • Add condition to check if wandb is installed before uninstalling it (link)

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

Enhanced dependency management for the yolov5 library in Jupyter Notebooks.

📊 Key Changes

  • Added a new check to verify if the wandb package is installed before uninstalling it.

🎯 Purpose & Impact

  • 🛠️ Purpose: To prevent automatic, undesired registration prompts from the wandb package during a yolov5 session in Jupyter Notebooks.
  • 🌐 Impact: Users will have a smoother experience with more control over their environment, avoiding potential hangs during initialization in notebooks.

@glenn-jocher glenn-jocher merged commit 2334aa7 into master Jun 18, 2023
@glenn-jocher glenn-jocher deleted the glenn-jocher-patch-1 branch June 18, 2023 14:09
@glenn-jocher glenn-jocher self-assigned this Jun 18, 2023
@glenn-jocher glenn-jocher added the fixed Bug has been resolved label Jun 18, 2023
bandakopi pushed a commit to irajcode/yolov5 that referenced this pull request Jul 20, 2023
* Uninstall `wandb` from notebook environments

Due to undesired behavior in https://www.kaggle.com/code/ultralytics/yolov8/comments#2306977

Signed-off-by: Glenn Jocher <[email protected]>

* fix import

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

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

---------

Signed-off-by: Glenn Jocher <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
pleb631 pushed a commit to pleb631/yolov5 that referenced this pull request Jan 6, 2024
* Uninstall `wandb` from notebook environments

Due to undesired behavior in https://www.kaggle.com/code/ultralytics/yolov8/comments#2306977

Signed-off-by: Glenn Jocher <[email protected]>

* fix import

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

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

---------

Signed-off-by: Glenn Jocher <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fixed Bug has been resolved
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant