Shunt fx_interpreter graphmodule print on error into tlparse#158469
Shunt fx_interpreter graphmodule print on error into tlparse#158469wconstab wants to merge 2 commits intogh/wconstab/427/basefrom
Conversation
Include both the error stacktrace and the graphmodule in a new structured trace artifact. Log the shortened version to the console, and also log a hint to look at the tlparse for more. [ghstack-poisoned]
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/158469
Note: Links to docs will display an error until the docs builds have been completed. ✅ You can merge normally! (1 Unrelated Failure)As of commit dabe26c with merge base 900fba4 ( UNSTABLE - The following job is marked as unstable, possibly due to flakiness on trunk:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
ezyang
left a comment
There was a problem hiding this comment.
I'd probably also check with whoever added this in the first place. Also, the error message is totally not going to help anyone who doesn't already know how to do this. @williamwen42 this is a perfect use case for the website BTW.
Looks like it was @bdhirsh - well, i'll go ahead and land it and when he's back we can confirm and change if needed.
I could link to a doc. Would this be the best link for docs? it at least shows usage. i am not sure if we have an official doc page or tutorial that shows this. |
Include both the error stacktrace and the graphmodule in a new structured trace artifact. Log the shortened version to the console, and also log a hint to look at the tlparse for more. cc ezyang SherlockNoMad EikanWang jgong5 wenzhe-nrv [ghstack-poisoned]
|
Starting merge as part of PR stack under #158481 |
1 similar comment
|
Starting merge as part of PR stack under #158481 |
Pull Request resolved: #158481 Approved by: https://github.com/d4l3k ghstack dependencies: #158469
Stack from ghstack (oldest at bottom):
Include both the error stacktrace and the graphmodule in a new
structured trace artifact. Log the shortened version to the console,
and also log a hint to look at the tlparse for more.
cc @ezyang @SherlockNoMad @EikanWang @jgong5 @wenzhe-nrv