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

Remove gitData script from docker build #2815

Merged
merged 1 commit into from
Jul 6, 2021

Conversation

dapplion
Copy link
Contributor

@dapplion dapplion commented Jul 6, 2021

Motivation

Completes #2812 removing a duplicate script from docker build

Description

Remove gitData script from docker build

@codeclimate
Copy link

codeclimate bot commented Jul 6, 2021

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

View more on Code Climate.

@github-actions
Copy link
Contributor

github-actions bot commented Jul 6, 2021

Performance Report

✔️ no performance regression detected

Full benchmark results
Benchmark suite Current: c9319a7 Previous: 04d8ef3 Ratio
getCommitteeAssignments - req 1000 vs - 250000 vc 9.6730 ms/op 8.3020 ms/op 1.17
Process block - 250000 vs - with 0 validator exit 249.54 us/op 323.73 us/op 0.77
Process block - 250000 vs - with 1 validator exit 21.476 ms/op 22.046 ms/op 0.97
Process block - 250000 vs - with 16 validator exits 22.305 ms/op 23.572 ms/op 0.95
epoch - 250000 vs - processJustificationAndFinalization 75.081 us/op 85.412 us/op 0.88
epoch - 250000 vs - processRewardsAndPenalties 296.61 ms/op 311.10 ms/op 0.95
epoch - 250000 vs - processRegistryUpdates 10.455 us/op 11.118 us/op 0.94
epoch - 250000 vs - processSlashings 37.616 us/op 48.441 us/op 0.78
epoch - 250000 vs - processFinalUpdates 43.133 ms/op 41.451 ms/op 1.04
epoch - 250000 vs - prepareEpochProcessState 571.85 ms/op 657.63 ms/op 0.87
getAttestationDeltas - 250000 vs 123.80 ms/op 118.91 ms/op 1.04
processSlots - 250000 vs - 32 empty slots 3.7810 s/op 3.9165 s/op 0.97
shuffle list - 512 els 162.71 us/op 167.05 us/op 0.97
shuffle list - 16384 els 1.9581 ms/op 1.8452 ms/op 1.06
shuffle list - 250000 els 28.007 ms/op 26.170 ms/op 1.07
aggregationBits - 2048 els - readonlyValues 262.98 us/op 265.80 us/op 0.99
aggregationBits - 2048 els - zipIndexesInBitList 51.733 us/op 56.894 us/op 0.91
ssz.Root.equals 1.4280 us/op 1.3020 us/op 1.10
getPubkeys - persistent - req 1000 vs - 200000 vc 19.077 us/op 17.175 us/op 1.11
BLS verify - blst-native 3.0611 ms/op 2.0047 ms/op 1.53
BLS verifyMultipleSignatures 3 - blst-native 4.4362 ms/op 4.2027 ms/op 1.06
BLS verifyMultipleSignatures 8 - blst-native 9.5830 ms/op 8.9415 ms/op 1.07
BLS verifyMultipleSignatures 32 - blst-native 36.329 ms/op 33.166 ms/op 1.10
BLS aggregatePubkeys 32 - blst-native 56.414 us/op 45.129 us/op 1.25
BLS aggregatePubkeys 128 - blst-native 180.66 us/op 176.76 us/op 1.02
CheckpointStateCache - add get delete 41.100 us/op 50.304 us/op 0.82
validate gossip signedAggregateAndProof - struct 5.2328 ms/op 5.2095 ms/op 1.00
validate gossip signedAggregateAndProof - treeBacked 5.2243 ms/op 5.1786 ms/op 1.01
validate gossip attestation - struct 2.4413 ms/op 2.3503 ms/op 1.04
validate gossip attestation - treeBacked 2.4948 ms/op 2.3996 ms/op 1.04

by benchmarkbot/action

@wemeetagain wemeetagain merged commit fde84b5 into master Jul 6, 2021
@wemeetagain wemeetagain deleted the dapplion/remove-git-data-script-docker branch July 6, 2021 18:24
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