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

Run benchmark actions on all push events #2779

Merged
merged 1 commit into from
Jun 30, 2021
Merged

Conversation

dapplion
Copy link
Contributor

Motivation

PR #2778 unintentionally disabled running the benchmark action on push events

Description

  • On PR events, run only if this repo
  • On push events, run always

@codeclimate
Copy link

codeclimate bot commented Jun 30, 2021

Code Climate has analyzed commit d14d939 and detected 0 issues on this pull request.

View more on Code Climate.

@github-actions
Copy link
Contributor

Performance Report

✔️ no performance regression detected

🚀🚀 Significant benchmark improvement detected

Benchmark suite Current: 554d994 Previous: bad3f6e Ratio
Process block - 250000 vs - with 0 validator exit 224.89 us/op 5.7165 ms/op 0.04
epoch - 250000 vs - processSlashings 34.312 us/op 3.5590 ms/op 0.01
epoch - 250000 vs - prepareEpochProcessState 534.58 ms/op 1.0793 s/op 0.50
shuffle list - 512 els 143.77 us/op 1.4239 ms/op 0.10
shuffle list - 16384 els 1.7125 ms/op 15.937 ms/op 0.11
shuffle list - 250000 els 24.546 ms/op 225.32 ms/op 0.11
Full benchmark results
Benchmark suite Current: 554d994 Previous: bad3f6e Ratio
Process block - 250000 vs - with 0 validator exit 224.89 us/op 5.7165 ms/op 0.04
Process block - 250000 vs - with 1 validator exit 20.339 ms/op 25.965 ms/op 0.78
Process block - 250000 vs - with 16 validator exits 20.945 ms/op 26.717 ms/op 0.78
epoch - 250000 vs - processJustificationAndFinalization 66.754 us/op 119.72 us/op 0.56
epoch - 250000 vs - processRewardsAndPenalties 272.38 ms/op 359.30 ms/op 0.76
epoch - 250000 vs - processRegistryUpdates 9.9850 us/op 10.317 us/op 0.97
epoch - 250000 vs - processSlashings 34.312 us/op 3.5590 ms/op 0.01
epoch - 250000 vs - processFinalUpdates 40.119 ms/op 39.089 ms/op 1.03
epoch - 250000 vs - prepareEpochProcessState 534.58 ms/op 1.0793 s/op 0.50
getAttestationDeltas - 250000 vs 118.25 ms/op 165.88 ms/op 0.71
processSlots - 250000 vs - 32 empty slots 3.5964 s/op 1.9698 s/op 1.83
shuffle list - 512 els 143.77 us/op 1.4239 ms/op 0.10
shuffle list - 16384 els 1.7125 ms/op 15.937 ms/op 0.11
shuffle list - 250000 els 24.546 ms/op 225.32 ms/op 0.11
aggregationBits - 2048 els - readonlyValues 218.12 us/op 273.96 us/op 0.80
aggregationBits - 2048 els - zipIndexesInBitList 46.539 us/op 56.178 us/op 0.83
ssz.Root.equals 1.2770 us/op 1.2810 us/op 1.00
BLS verify - blst-native 2.0000 ms/op
BLS verifyMultipleSignatures 3 - blst-native 3.8614 ms/op
BLS verifyMultipleSignatures 8 - blst-native 8.4833 ms/op
BLS verifyMultipleSignatures 32 - blst-native 32.393 ms/op
BLS aggregatePubkeys 32 - blst-native 41.344 us/op
BLS aggregatePubkeys 128 - blst-native 164.87 us/op
CheckpointStateCache - add get delete 33.299 us/op

by benchmarkbot/action

@dapplion dapplion merged commit db5d153 into master Jun 30, 2021
@dapplion dapplion deleted the dapplion/fix-benchmark-if branch June 30, 2021 15:28
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.

2 participants