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

feature: Add flag to dump debug t8n information #219

Merged
merged 6 commits into from
Jul 25, 2023

Conversation

marioevz
Copy link
Member

Adds flag --transition-tool-debug-dump-path which can be used to specify a path where debugging information for all transition tool invocations will be dumped.

For example:

fill ./tests/frontier/opcodes/test_dup.py  --transition-tool-debug-dump-path=./tmp/

Will result in the following folders:

test_dup_fork_Berlin             test_dup_fork_Frontier   test_dup_fork_Merge
test_dup_fork_Byzantium          test_dup_fork_Homestead  test_dup_fork_Shanghai
...

inside ./tmp/, and inside each folder there will be several numbered subfolders with all the information about each call to the transition tool.

@shemnon @chfast let me know if this is helpful and whether you have any suggestions!

Copy link
Collaborator

@spencer-tb spencer-tb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Works well on my end - I like t8n_dump_dir :D

@marioevz marioevz merged commit 68eb27a into ethereum:main Jul 25, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants