-
Notifications
You must be signed in to change notification settings - Fork 59
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(fw): add pretty printing of vm traces #433
Comments
I think can even optimise it better. I have 2 options --vmtraceraw is always available. (Unformatted) |
Can we use this package for formatting the debug trace? |
Hi @raxhvl, looks like a nice package. Would you like to pick this up? You may want to wait until #474 is merged, or base your work on that. |
Hi @raxhvl thanks! If you check the branch on #474, compile the required geth branch, then inject an error in any test (I've been dropping the gas limit in this test to produce an Out-of-gas error for example), you can see the current output of the relevant traces for the test, which are very bland. If you'd like to give it a go to use rich package to print this a bit better it would be very nice. |
Currently,
fill --traces -s
does print vm traces to the terminal if a test fails (note-s
to prevent pytest from capturing stdout), but they're rather ugly. It'd be nice to have a more compact pretty print of these traces.For example (assuming the test has been modified to fail upon filling):
fill tests/berlin/eip2930_access_list/ --fork Berlin --evm-dump-dir=/tmp/evm-dump --traces -s
Screenshot:
Here's an example of a clean trace representation from retesteth:
The text was updated successfully, but these errors were encountered: