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
Refactor generated benchmark files to allow for more general usage #4436
Conversation
…refactor-files
…refactor-files
@0xKiwi lint fails |
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions. |
Codecov Report
@@ Coverage Diff @@
## master #4436 +/- ##
=========================================
Coverage ? 58.57%
=========================================
Files ? 236
Lines ? 16114
Branches ? 0
=========================================
Hits ? 9438
Misses ? 5301
Partials ? 1375 |
…refactor-files
…refactor-files
shared/benchutil/pregen.go
Outdated
|
||
// ValidatorCount is for declaring how many validators the benchmarks will be | ||
// performed with. Default is 65536 or 2M ETH staked. | ||
var ValidatorCount = uint64(65536) |
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.
Can you change this to genesis validator count?
…refactor-files
…rysmaticlabs#4436) * Begin to refactor benchmark files to testutil * Complete most of refactoring * Fix file path * gofmt * Fix path * Move generatego to tools/ * Move gen util to tools/benchmark-files-gen * Add comments to pregen funcs * Make function names consistent * Update README * Redo benchmarks with 16384 validators
…rysmaticlabs#4436) * Begin to refactor benchmark files to testutil * Complete most of refactoring * Fix file path * gofmt * Fix path * Move generatego to tools/ * Move gen util to tools/benchmark-files-gen * Add comments to pregen funcs * Make function names consistent * Update README * Redo benchmarks with 16384 validators
This PR takes the SSZ files used for benchmarking and moves them to their own
benchutil
folder.It also moves the tool to generate the benchmark files to
tools/benchmark-files-gen
, with added documentation to generate the files if needed.Now any benchmarking tests can just use the pregen state within
benchutil
:)