-
Notifications
You must be signed in to change notification settings - Fork 618
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
feat(sdk): always print the traceback inside of the wandb.init
context manager
#4603
Conversation
wandb.init
context manager
wandb.init
context managerwandb.init
context manager
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #4603 +/- ##
==========================================
- Coverage 77.87% 77.80% -0.08%
==========================================
Files 379 379
Lines 43746 43749 +3
==========================================
- Hits 34069 34039 -30
- Misses 9624 9657 +33
Partials 53 53
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Any chance to merge this tiny change? It's apparently been a problem for two years! (#2387) We just started our W&B trial and hit it on day 3. And the PR is right here! I've patched it in and it works great! I tried sweeps from a notebook, and I was getting errors only in the sweep but not with normal runs. It looks like sweeps run in a separate process, so I can't use shared data. Which is fine. But this is probably a common way to get sweep-only issues, and they are painful to debug without the stack trace. Thanks for the PR! 👍 |
After this awesome request, we can't say no. So we will include it in the upcoming release. 😄 |
Yes!!! Can't wait, thanks! |
Fixes WB-9886
Fixes WB-10858
Fixes #4225
Description
What does the PR do?
Testing
How was this PR tested?
Checklist