-
-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
Pass traceback to ServerConnection.ResponseError #14328
Pass traceback to ServerConnection.ResponseError #14328
Conversation
Thanks for making a pull request to jupyterlab! |
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.
Thanks @a3626a I took the liberty to push an improvement to preserve default values.
My previous implementation preserves default values.
However I don't care about details of implmentation. I'm 100% satisfied if the traceback is passed anyway. Thank you for your feedback and tests. |
Thanks @a3626a I learnt something - I was not aware that explicitly passed |
@meeseeksdev please backport to 3.6.x |
Benchmark reportThe execution time (in milliseconds) are grouped by test file, test type and browser. The mean relative comparison is computed with 95% confidence. Results table
Changes are computed with expected as reference.
Waiting for localhost:8888 Cell memory leaksCreate a code cellMemory change: +148 kB Leak detected: YesLeaking objects:
Leaking collections: Create a markdown cellMemory change: -104 kB Leak detected: NoLeaking objects:
Leaking collections: Create a raw cellMemory change: -176 kB Leak detected: NoLeaking objects:
Leaking collections:
File editor memory leaksCreate a fileMemory change: -81.3 kB Leak detected: NoLeaking objects:
Notebook memory leaksCreate a notebookMemory change: +34.3 kB Leak detected: YesLeaking objects:
2 passing (7m)
|
…14368) Co-authored-by: ChangHwan Lee / 이창환 <[email protected]>
In my Jupyter Lab environment,
Error: Unhandled Error
is reported by monitoring tool. I'm investigating each errors and making some handling for them. HoweverError: Unhandled Error
does not have enough infomation to be handled properly. On jupyter lab side, I found thatServerConnection.ResponseError
has a room for server sidetraceback
, but it is not well managed. Iftrackback
works properly, it can give us enough infomation for errors to be handled.(It is reported as
Error
, but actuallyServerConnection.ResponseError
)References
Code changes
Pass
trackback
to ServerConnection.ResponseError. I think it is intended behavior of ResponseError at first, but looks like missing somehow.I wrote the code in a conservative manner to preserve previous behavior as much I can. That is why
data['message'] ? data['message'] : undefined
comes.User-facing changes
No
Backwards-incompatible changes
Actually, I found no code that uses
traceback
of the ResponseError constructor. It is backword compatible.