-
Notifications
You must be signed in to change notification settings - Fork 413
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
Adding log exception to Mosaic Logger #3089
Conversation
Can we do some sort of manual test to see how the exception looks when it's logged to mcloud? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How is log exception called?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
whoa that manual test example is pretty cool. I like that it shows up really nicely in the describe run output. LGTM. Thanks, @jjanezhang !
* adding failure reason log wrapper * added test and exception to json * removed non custom class vars from run metadata exc * force flush exceptions
Adding log exception to Mosaic Logger
Adding function to log exceptions to run metadata
Being used in foundry to catch custom errors
Manually tested:
Related PR: mosaicml/llm-foundry#1014