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

Merged
merged 1 commit into from
Jun 18, 2023
Merged

Conversation

glenn-jocher
Copy link
Member

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

Resolves unwanted W&B install issues in https://www.kaggle.com/code/ultralytics/yolov8/comments#2306977

🤖 Generated by Copilot at dcaa6db

Summary

🗑️🚫🛠️

Uninstall wandb package in utils/__init__.py to avoid conflicts with Colab. This affects the notebook_init function that prepares the YOLOv5 notebook.

wandb uninstalled
Colab has it already
notebook init changed

Walkthrough

  • Uninstall wandb package to avoid conflicts with Colab environment (link)

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

Improvements to notebook initialization in YOLOv5 by ensuring the WandB is uninstalled in certain environments.

📊 Key Changes

  • Added a command to uninstall the wandb Python package during the environment setup process.

🎯 Purpose & Impact

  • 🎯 Purpose: The main goal seems to cater specifically to environments like Google Colab, where a clean slate is desired for certain dependencies when initializing notebooks. By explicitly uninstalling wandb, it may help avoid potential conflicts or bugs related to WandB that may already be installed in the environment.
  • 🎯 Impact: This change will ensure that whenever a user starts up a new instance on platforms like Colab, they won't have wandb installed by default, potentially avoiding version mismatches and streamlining the setup process. Users looking to use wandb will need to reinstall it, which could add an extra step, but ensures that they have the latest and compatible version.

@glenn-jocher
Copy link
Member Author

@AyushExel problem solved :)

@glenn-jocher glenn-jocher merged commit f616dc5 into master Jun 18, 2023
@glenn-jocher glenn-jocher deleted the remove_wandb branch June 18, 2023 13:44
bandakopi pushed a commit to irajcode/yolov5 that referenced this pull request Jul 20, 2023
Uninstall W&B that are present in notebooks

Resolves unwanted W&B install issues in https://www.kaggle.com/code/ultralytics/yolov8/comments#2306977

Signed-off-by: Glenn Jocher <[email protected]>
pleb631 pushed a commit to pleb631/yolov5 that referenced this pull request Jan 6, 2024
Uninstall W&B that are present in notebooks

Resolves unwanted W&B install issues in https://www.kaggle.com/code/ultralytics/yolov8/comments#2306977

Signed-off-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.

1 participant