-
-
Notifications
You must be signed in to change notification settings - Fork 16.6k
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
Conversation
for more information, see https://pre-commit.ci
@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. |
@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. |
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
@AyushExel thanks! PR is merged. Thank you for your contributions to YOLOv5 🚀 and Vision AI ⭐ |
* 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 <[email protected]>
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
0.12.11
or newer.0.12.10
of wandb or below for full compatibility.🎯 Purpose & Impact