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

DO-2043 / use ephemeral github runner based on EC2 instance #1657

Merged
17 commits merged into from
Dec 1, 2023
Merged

Conversation

ghost
Copy link

@ghost ghost commented Nov 28, 2023

Summary

INSTRUCTIONS:
Use github runner based on EC2 instance to save money

Details

INSTRUCTIONS:
Run a workflow.

Testing

INSTRUCTIONS:
Run a workflow

Copy link

github-actions bot commented Nov 29, 2023

Benchmark for 1c6efb6

Click to view benchmark
Test Base PR %
costing::bench_prepare_wasm 65.0±0.54ms 65.4±0.61ms +0.62%
costing::decode_sbor 13.1±0.02µs 13.2±0.02µs +0.76%
costing::decode_sbor_bytes 37.0±0.03µs 36.4±0.03µs -1.62%
costing::deserialize_wasm 1254.6±1.17µs 1263.7±1.44µs +0.73%
costing::instantiate_flash_loan 4.0±0.45ms 4.0±0.50ms 0.00%
costing::instantiate_radiswap 5.2±0.06ms 5.2±0.08ms 0.00%
costing::spin_loop 23.1±0.01ms 23.2±0.01ms +0.43%
costing::validate_sbor_payload 25.7±0.02µs 25.8±0.30µs +0.39%
costing::validate_sbor_payload_bytes 342.4±4.38ns 358.1±0.24ns +4.59%
costing::validate_secp256k1 80.5±0.06µs 80.4±0.10µs -0.12%
costing::validate_wasm 35.7±0.06ms 35.8±0.06ms +0.28%
decimal::add/0 7.2±0.00ns 7.2±0.00ns 0.00%
decimal::add/rust-native 9.1±0.05ns 9.1±0.04ns 0.00%
decimal::add/wasmer 138.7±0.03ns 139.5±0.26ns +0.58%
decimal::add/wasmer-call-native 588.7±0.17ns 590.6±0.30ns +0.32%
decimal::add/wasmi 460.5±1.67ns 470.1±0.06ns +2.08%
decimal::add/wasmi-call-native 5.8±0.01µs 5.9±0.03µs +1.72%
decimal::div/0 166.5±0.19ns 165.6±0.08ns -0.54%
decimal::from_string/0 145.5±0.20ns 145.3±0.04ns -0.14%
decimal::mul/0 129.0±0.09ns 128.9±0.09ns -0.08%
decimal::mul/rust-native 131.9±0.08ns 131.9±0.20ns 0.00%
decimal::mul/wasmer 1711.4±0.54ns 1720.1±3.10ns +0.51%
decimal::mul/wasmer-call-native 717.6±0.45ns 721.8±0.27ns +0.59%
decimal::mul/wasmi 30.8±0.03µs 30.0±0.04µs -2.60%
decimal::mul/wasmi-call-native 6.0±0.02µs 6.0±0.03µs 0.00%
decimal::pow/0 618.9±0.09ns 632.4±0.48ns +2.18%
decimal::pow/rust-native 605.8±0.32ns 608.0±0.28ns +0.36%
decimal::pow/wasmer 7.5±0.00µs 7.6±0.00µs +1.33%
decimal::pow/wasmer-call-native 1126.3±0.62ns 1123.6±0.31ns -0.24%
decimal::pow/wasmi 144.4±0.22µs 160.6±0.17µs +11.22%
decimal::pow/wasmi-call-native 5.6±0.04µs 5.7±0.04µs +1.79%
decimal::root/0 9.0±0.00µs 9.1±0.00µs +1.11%
decimal::sub/0 7.2±0.00ns 7.2±0.00ns 0.00%
decimal::to_string/0 493.9±0.21ns 494.4±0.24ns +0.10%
precise_decimal::add/0 8.7±0.00ns 8.0±0.00ns -8.05%
precise_decimal::add/rust-native 10.4±0.03ns 10.4±0.02ns 0.00%
precise_decimal::add/wasmer 145.5±0.08ns 146.8±0.11ns +0.89%
precise_decimal::add/wasmer-call-native 604.0±0.26ns 609.9±0.19ns +0.98%
precise_decimal::add/wasmi 559.7±1.24ns 560.0±0.42ns +0.05%
precise_decimal::add/wasmi-call-native 6.2±0.02µs 6.3±0.03µs +1.61%
precise_decimal::div/0 263.8±0.09ns 264.6±0.34ns +0.30%
precise_decimal::from_string/0 202.3±0.08ns 202.2±0.42ns -0.05%
precise_decimal::mul/0 283.2±0.16ns 283.7±0.15ns +0.18%
precise_decimal::mul/rust-native 262.3±0.09ns 261.2±0.07ns -0.42%
precise_decimal::mul/wasmer 4.0±0.00µs 4.0±0.00µs 0.00%
precise_decimal::mul/wasmer-call-native 952.2±0.40ns 904.0±0.31ns -5.06%
precise_decimal::mul/wasmi 79.3±0.06µs 78.3±0.03µs -1.26%
precise_decimal::mul/wasmi-call-native 6.6±0.02µs 6.6±0.02µs 0.00%
precise_decimal::pow/0 1572.0±0.65ns 1572.5±0.66ns +0.03%
precise_decimal::pow/rust-native 1284.3±0.29ns 1266.9±0.60ns -1.35%
precise_decimal::pow/wasmer 18.9±0.01µs 18.9±0.01µs 0.00%
precise_decimal::pow/wasmer-call-native 2.1±0.00µs 2.1±0.00µs 0.00%
precise_decimal::pow/wasmi 383.6±0.23µs 379.4±0.25µs -1.09%
precise_decimal::pow/wasmi-call-native 10.9±0.04µs 11.0±0.03µs +0.92%
precise_decimal::root/0 61.4±0.02µs 61.7±0.08µs +0.49%
precise_decimal::sub/0 8.9±0.01ns 8.3±0.00ns -6.74%
precise_decimal::to_string/0 754.0±0.18ns 747.0±0.16ns -0.93%
schema::validate_payload 281.0±0.18µs 280.9±0.23µs -0.04%
transaction::radiswap 5.4±0.03ms 5.5±0.04ms +1.85%
transaction::transfer 1753.1±2.84µs 1763.9±4.02µs +0.62%
transaction_processing::prepare 2.5±0.00ms 2.5±0.00ms 0.00%
transaction_processing::prepare_and_decompile 6.4±0.01ms 6.4±0.01ms 0.00%
transaction_processing::prepare_and_decompile_and_recompile 24.0±0.18ms 24.3±0.26ms +1.25%
transaction_validation::validate_manifest 44.1±0.07µs 44.2±0.03µs +0.23%
transaction_validation::verify_ecdsa 78.0±0.05µs 78.0±0.04µs 0.00%
transaction_validation::verify_ed25519 52.1±0.04µs 52.1±0.19µs 0.00%

@ghost ghost marked this pull request as ready for review November 29, 2023 13:58
.github/workflows/ci.yml Outdated Show resolved Hide resolved
@ghost ghost requested a review from lrubasze November 30, 2023 15:21
uses: jwlawson/[email protected]
with:
cmake-version: '3.27.9'
- name: Cargo Check
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor thing, but the step name seems irrelevant.

@ghost ghost merged commit 34e9b24 into develop Dec 1, 2023
4 checks passed
@ghost ghost deleted the DO-2043 branch May 22, 2024 12:40
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant