-
Notifications
You must be signed in to change notification settings - Fork 760
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
Use benchmark.js #830
Use benchmark.js #830
Conversation
configure rhysd/github-action-benchmark
Codecov Report
@@ Coverage Diff @@
## vm/benchmark #830 +/- ##
================================================
- Coverage 84.42% 84.23% -0.20%
================================================
Files 16 17 +1
Lines 1239 1224 -15
Branches 247 246 -1
================================================
- Hits 1046 1031 -15
Misses 125 125
Partials 68 68
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
233e851
to
b402204
Compare
Super cool, the tool suite you figured out with this |
b402204
to
5987454
Compare
5987454
to
cc7b107
Compare
This is now ready for review to merge into #794 and then would like to merge into master. |
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.
Looks good
This PR targets merging into #794 to:
vm.runBlock()
instead of looping over txs withvm.runTx()
Sample output:
Pages link: http://ethereumjs.github.io/ethereumjs-vm/dev/bench/