ChainSafe / blst-ts

Typescript wrapper for https://github.com/supranational/blst native bindings, a highly performant BLS12-381 signature library
Other
18 stars 13 forks source link

fix: update workflow for new actions version #132

Closed matthewkeil closed 6 months ago

matthewkeil commented 6 months ago

Description

I updated the actions/upload-artifacts and actions/download-artifacts and the API changed slightly so it didn't work the way it used to in the old workflow.

There were a couple of other small things that I noticed during publishing

github-actions[bot] commented 6 months ago

Performance Report

✔️ no performance regression detected

Full benchmark results | Benchmark suite | Current: 9873fa89606e21b635e8f6dae42f21780fc6d501 | Previous: - | Ratio | |-|-|-|-| | PublicKey serialization | 1.2270 us/op | | PublicKey deserialize | 22.793 us/op | | PublicKey deserialize and validate - 1 keys | 84.015 us/op | | PublicKey deserialize and validate - 100 keys | 8.2694 ms/op | | PublicKey deserialize and validate - 10000 keys | 835.54 ms/op | | SecretKey.fromKeygen | 3.6270 us/op | | SecretKey serialization | 1.0670 us/op | | SecretKey deserialization | 1.4950 us/op | | SecretKey.toPublicKey | 138.13 us/op | | SecretKey.sign | 532.01 us/op | | Signature serialization | 1.3620 us/op | | Signature deserialize | 44.513 us/op | | Signatures deserialize and validate - 1 sets | 120.43 us/op | | Signatures deserialize and validate - 100 sets | 11.934 ms/op | | Signatures deserialize and validate - 10000 sets | 1.1970 s/op | | aggregatePublicKeys - 1 sets | 2.4920 us/op | | aggregatePublicKeys - 8 sets | 10.497 us/op | | aggregatePublicKeys - 32 sets | 37.818 us/op | | aggregatePublicKeys - 128 sets | 147.02 us/op | | aggregatePublicKeys - 256 sets | 294.34 us/op | | aggregateSignatures - 1 sets | 4.1790 us/op | | aggregateSignatures - 8 sets | 24.488 us/op | | aggregateSignatures - 32 sets | 91.968 us/op | | aggregateSignatures - 128 sets | 361.39 us/op | | aggregateSignatures - 256 sets | 724.43 us/op | | aggregateVerify - 1 sets | 1.5879 ms/op | | aggregateVerify - 8 sets | 5.7680 ms/op | | aggregateVerify - 32 sets | 20.560 ms/op | | aggregateVerify - 128 sets | 79.237 ms/op | | aggregateVerify - 256 sets | 157.75 ms/op | | verifyMultipleAggregateSignatures - 1 sets | 1.6403 ms/op | | verifyMultipleAggregateSignatures - 8 sets | 6.1730 ms/op | | verifyMultipleAggregateSignatures - 32 sets | 22.104 ms/op | | verifyMultipleAggregateSignatures - 128 sets | 85.982 ms/op | | verifyMultipleAggregateSignatures - 256 sets | 170.97 ms/op | | Same message - 1 sets | 1.7115 ms/op | | Same message - 8 sets | 2.5811 ms/op | | Same message - 32 sets | 5.5287 ms/op | | Same message - 128 sets | 17.283 ms/op | | Same message - 256 sets | 33.090 ms/op | | libuv multithreading - addVerificationRandomness true | 20.633 s/op | | libuv multithreading - addVerificationRandomness false | 20.656 s/op |

by benchmarkbot/action

matthewkeil commented 6 months ago

Example of a successful run. Test code removed in a42c701

Screenshot 2024-04-09 at 9 50 31 PM