Skip to content
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

[13.0] [IMP] Statechart interpreter: print execution traceback #38

Open
wants to merge 3 commits into
base: 13.0
Choose a base branch
from

Conversation

ThomasBinsfeld
Copy link
Member

In some cases, statechart hide the root traceback of an error, making it very difficult to debug.

The raise statement raise _root_cause(e).with_traceback(sys.exc_info()[2]) may be improved but raise _root_cause(e) is not a good solution.

While waiting for a better solution that would allow to raise an error with the complete traceback, let's print additionnal information with traceback.print_exc(). We're now sure to have everything needed for debug.

In some cases, statechart hide the source traceback of an error.
Print the execution traceback to be sure to have all details in log.
@sbidoul
Copy link
Member

sbidoul commented Dec 15, 2023

@ThomasBinsfeld @sbejaoui in 14 and 16 it now does raise _root_cause()... from e. This should preserve the original exception. What do you think?

raise _root_cause(e).with_traceback(sys.exc_info()[2]) from e

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants