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

Feature: Remove replay sync #1643

Closed
wants to merge 1 commit into from
Closed

Conversation

talekhinezh
Copy link
Member

Moved tool into ledger-tools

@talekhinezh talekhinezh changed the base branch from main to develop November 6, 2023 21:17
@talekhinezh talekhinezh marked this pull request as ready for review November 6, 2023 21:17
Copy link

github-actions bot commented Nov 6, 2023

Benchmark for 1708154

Click to view benchmark
Test Base PR %
costing::bench_prepare_wasm 76.8±0.39ms 76.7±0.98ms -0.13%
costing::decode_sbor 17.0±0.14µs 16.9±0.12µs -0.59%
costing::decode_sbor_bytes 54.3±0.26µs 54.4±0.29µs +0.18%
costing::deserialize_wasm 1388.7±7.12µs 1404.3±22.41µs +1.12%
costing::instantiate_flash_loan 4.6±0.77ms 4.9±1.05ms +6.52%
costing::instantiate_radiswap 7.0±0.16ms 7.2±0.15ms +2.86%
costing::spin_loop 26.1±0.03ms 26.2±0.03ms +0.38%
costing::validate_sbor_payload 30.4±0.04µs 30.8±0.09µs +1.32%
costing::validate_sbor_payload_bytes 399.7±0.65ns 377.9±0.70ns -5.45%
costing::validate_secp256k1 87.6±0.49µs 87.6±0.13µs 0.00%
costing::validate_wasm 43.1±0.12ms 44.1±0.17ms +2.32%
decimal::add/0 9.3±0.00ns 9.3±0.00ns 0.00%
decimal::add/rust-native 10.9±0.02ns 10.9±0.01ns 0.00%
decimal::add/wasmer 132.2±0.20ns 155.2±0.44ns +17.40%
decimal::add/wasmer-call-native 570.0±1.44ns 576.5±0.43ns +1.14%
decimal::add/wasmi 835.1±4.61ns 948.3±8.23ns +13.56%
decimal::add/wasmi-call-native 8.0±0.01µs 8.7±0.01µs +8.75%
decimal::div/0 199.9±0.44ns 200.1±0.37ns +0.10%
decimal::from_string/0 163.4±0.43ns 162.5±0.50ns -0.55%
decimal::mul/0 154.5±0.25ns 156.3±0.31ns +1.17%
decimal::mul/rust-native 155.4±0.13ns 149.9±0.14ns -3.54%
decimal::mul/wasmer 1679.6±0.52ns 1687.8±0.78ns +0.49%
decimal::mul/wasmer-call-native 699.7±0.30ns 796.3±0.49ns +13.81%
decimal::mul/wasmi 60.3±0.06µs 63.4±0.45µs +5.14%
decimal::mul/wasmi-call-native 8.1±0.00µs 8.9±0.01µs +9.88%
decimal::pow/0 725.7±0.78ns 723.1±1.31ns -0.36%
decimal::pow/rust-native 700.8±1.03ns 701.3±1.06ns +0.07%
decimal::pow/wasmer 7.3±0.02µs 7.7±0.01µs +5.48%
decimal::pow/wasmer-call-native 1184.2±0.49ns 1181.0±0.68ns -0.27%
decimal::pow/wasmi 283.6±0.29µs 302.4±0.79µs +6.63%
decimal::pow/wasmi-call-native 7.2±0.01µs 7.8±0.01µs +8.33%
decimal::root/0 8.9±0.01µs 9.7±0.02µs +8.99%
decimal::sub/0 9.4±0.01ns 9.4±0.01ns 0.00%
decimal::to_string/0 477.7±0.41ns 486.0±1.55ns +1.74%
precise_decimal::add/0 10.5±0.06ns 10.2±0.06ns -2.86%
precise_decimal::add/rust-native 12.7±0.00ns 12.7±0.00ns 0.00%
precise_decimal::add/wasmer 137.3±3.89ns 155.8±0.38ns +13.47%
precise_decimal::add/wasmer-call-native 573.3±0.78ns 609.3±3.07ns +6.28%
precise_decimal::add/wasmi 1031.1±4.27ns 1178.3±5.10ns +14.28%
precise_decimal::add/wasmi-call-native 8.5±0.01µs 9.1±0.04µs +7.06%
precise_decimal::div/0 326.8±1.23ns 326.1±0.57ns -0.21%
precise_decimal::from_string/0 224.7±0.14ns 221.2±0.32ns -1.56%
precise_decimal::mul/0 348.1±0.82ns 352.4±2.41ns +1.24%
precise_decimal::mul/rust-native 326.2±0.40ns 325.5±1.60ns -0.21%
precise_decimal::mul/wasmer 3.9±0.00µs 3.9±0.01µs 0.00%
precise_decimal::mul/wasmer-call-native 945.1±1.95ns 955.7±1.16ns +1.12%
precise_decimal::mul/wasmi 150.9±0.09µs 161.8±0.33µs +7.22%
precise_decimal::mul/wasmi-call-native 8.9±0.01µs 9.6±0.01µs +7.87%
precise_decimal::pow/0 1944.8±3.80ns 1982.1±18.55ns +1.92%
precise_decimal::pow/rust-native 1570.6±3.22ns 1621.9±7.74ns +3.27%
precise_decimal::pow/wasmer 18.2±0.02µs 18.3±0.04µs +0.55%
precise_decimal::pow/wasmer-call-native 2.3±0.00µs 2.4±0.01µs +4.35%
precise_decimal::pow/wasmi 736.0±6.89µs 771.8±1.21µs +4.86%
precise_decimal::pow/wasmi-call-native 16.6±0.05µs 17.3±0.03µs +4.22%
precise_decimal::root/0 64.2±0.03µs 65.0±0.12µs +1.25%
precise_decimal::sub/0 10.5±0.02ns 10.6±0.11ns +0.95%
precise_decimal::to_string/0 770.3±2.66ns 783.2±2.95ns +1.67%
schema::validate_payload 355.3±1.24µs 356.0±0.97µs +0.20%
transaction::radiswap 6.6±0.07ms 6.3±0.09ms -4.55%
transaction::transfer 2.1±0.01ms 2.0±0.01ms -4.76%
transaction_processing::prepare 2.9±0.01ms 2.9±0.01ms 0.00%
transaction_processing::prepare_and_decompile 7.5±0.10ms 7.2±0.02ms -4.00%
transaction_processing::prepare_and_decompile_and_recompile 27.5±0.24ms 26.7±0.16ms -2.91%
transaction_validation::validate_manifest 48.7±0.03µs 48.7±0.14µs 0.00%
transaction_validation::verify_ecdsa 85.1±0.09µs 85.8±5.56µs +0.82%
transaction_validation::verify_ed25519 60.8±0.09µs 60.6±0.25µs -0.33%

Copy link
Member

@iamyulong iamyulong left a comment

Choose a reason for hiding this comment

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

Please do not merge before comments on slack are resolved.

@talekhinezh talekhinezh closed this Nov 7, 2023
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.

3 participants