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

Add version warning for wandb #7385

Merged
merged 7 commits into from
Apr 12, 2022
Merged

Conversation

AyushExel
Copy link
Contributor

@AyushExel AyushExel commented Apr 12, 2022

This PR adds warning if wandb version is higher than 0.12.10. There's a regression causing resume from artifacts to not work in versions greater than 0.12.10. While we get it fixed in the client, it's better to warn users.

Fixes - #7077

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

Enhanced logging to warn users of compatibility issues with newer wandb versions 🚦

📊 Key Changes

  • Added a warning message that triggers if the wandb version is 0.12.11 or newer.
  • The warning informs users that YOLOv5 currently requires version 0.12.10 of wandb or below for full compatibility.

🎯 Purpose & Impact

  • This update is aimed at preventing potential issues that users may encounter when using incompatible versions of wandb with YOLOv5.
  • Users will benefit from knowing they should stick to an earlier version of wandb to ensure all features of YOLOv5 work as expected. This proactive communication can save time troubleshooting and preserve a smooth user experience. 🛠️

@glenn-jocher
Copy link
Member

@AyushExel thanks for the PR!

Can you wake up Lavanya and have her respond to my emails please? I've sent her several messages over the last few weeks with no response at all.

@AyushExel
Copy link
Contributor Author

@glenn-jocher thanks! I'll reach out to her. She's actually working from a different location temporarily and isn't responding to most mails. I'll remind her to get back to you.

@glenn-jocher glenn-jocher merged commit 74aaab3 into ultralytics:master Apr 12, 2022
@glenn-jocher
Copy link
Member

@AyushExel thanks! 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
* add version warning

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

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

* Cleanup

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

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

* Update __init__.py

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

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

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
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.

None yet

2 participants