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.

Sign up for GitHub

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

Jump to bottom

feat: add ci for scrypto-dev-container docker target #2034

Merged
merged 2 commits into from
Jan 6, 2025
Merged

feat: add ci for scrypto-dev-container docker target #2034

merged 2 commits into from
Jan 6, 2025

Conversation

azizi-a
Copy link
Contributor

@azizi-a azizi-a commented Dec 3, 2024 •
edited
Loading

Summary

Add CI to build and publish scrypto-dev-container docker target

@github-actions GitHub Actions
Copy link

github-actions bot commented Dec 3, 2024 •
edited
Loading

Docker tags
docker.io/radixdlt/private-scrypto-builder:d4030f45b7

@github-actions GitHub Actions
Copy link

github-actions bot commented Dec 3, 2024 •
edited
Loading

Docker tags
docker.io/radixdlt/private-scrypto-dev-container:d4030f45b7

@github-actions GitHub Actions
Copy link

github-actions bot commented Dec 3, 2024 •
edited
Loading

Benchmark for d4030f4

Click to view benchmark
Test Base PR %
costing::bench_prepare_wasm 44.4±0.16ms 44.5±0.20ms +0.23%
costing::decode_encoded_i8_array_to_manifest_raw_value 19.3±0.02ms 19.7±0.06ms +2.07%
costing::decode_encoded_i8_array_to_manifest_value 40.6±0.05ms 41.5±0.10ms +2.22%
costing::decode_encoded_tuple_array_to_manifest_raw_value 70.5±0.10ms 71.3±0.14ms +1.13%
costing::decode_encoded_tuple_array_to_manifest_value 116.1±1.12ms 98.5±0.14ms -15.16%
costing::decode_encoded_u8_array_to_manifest_raw_value 25.8±0.05µs 25.7±0.15µs -0.39%
costing::decode_encoded_u8_array_to_manifest_value 40.7±0.05ms 41.5±0.08ms +1.97%
costing::decode_rpd_to_manifest_raw_value 14.7±0.03µs 14.7±0.02µs 0.00%
costing::decode_rpd_to_manifest_value 10.8±0.03µs 10.8±0.07µs 0.00%
costing::deserialize_wasm 1215.4±2.47µs 1207.8±3.64µs -0.63%
costing::execute_transaction_creating_big_vec_substates 696.5±3.11ms 691.1±11.38ms -0.78%
costing::execute_transaction_reading_big_vec_substates 598.9±1.92ms 594.7±0.98ms -0.70%
costing::instantiate_flash_loan 924.3±772.92µs 997.5±869.40µs +7.92%
costing::instantiate_radiswap 1005.5±1022.71µs 973.8±760.76µs -3.15%
costing::scrypto_malloc 682.9±2.48ms 676.2±1.52ms -0.98%
costing::scrypto_sbor_decode 663.2±1.08ms 671.0±1.03ms +1.18%
costing::scrypto_sha256 584.2±1.06ms 583.7±1.29ms -0.09%
costing::spin_loop_v1 513.5±0.81ms 510.2±1.13ms -0.64%
costing::spin_loop_v2 595.8±4.71ms 594.8±0.60ms -0.17%
costing::validate_sbor_payload 29.2±0.05µs 29.2±0.07µs 0.00%
costing::validate_sbor_payload_bytes 247.8±0.56ns 249.2±0.41ns +0.56%
costing::validate_secp256k1 76.6±0.07µs 76.6±0.05µs 0.00%
costing::validate_wasm 33.5±0.05ms 33.4±0.03ms -0.30%
decimal::add/0 8.4±0.00ns 8.4±0.00ns 0.00%
decimal::add/rust-native 9.8±0.00ns 9.8±0.00ns 0.00%
decimal::add/wasmi 320.4±1.27ns 317.7±1.77ns -0.84%
decimal::add/wasmi-call-native 3.0±0.01µs 3.1±0.01µs +3.33%
decimal::div/0 175.0±0.39ns 174.4±0.59ns -0.34%
decimal::from_string/0 157.2±0.16ns 156.2±0.22ns -0.64%
decimal::mul/0 129.2±0.09ns 128.5±0.29ns -0.54%
decimal::mul/rust-native 124.9±0.05ns 124.9±0.03ns 0.00%
decimal::mul/wasmi 18.3±0.04µs 18.4±0.05µs +0.55%
decimal::mul/wasmi-call-native 3.1±0.01µs 3.2±0.01µs +3.23%
decimal::pow/0 592.1±0.37ns 590.7±0.50ns -0.24%
decimal::pow/rust-native 590.1±0.26ns 590.0±0.32ns -0.02%
decimal::pow/wasmi 88.4±0.15µs 89.1±0.17µs +0.79%
decimal::pow/wasmi-call-native 4.7±0.00µs 4.8±0.01µs +2.13%
decimal::root/0 8.2±0.01µs 8.2±0.01µs 0.00%
decimal::sub/0 8.4±0.01ns 8.4±0.01ns 0.00%
decimal::to_string/0 442.5±0.44ns 443.6±0.32ns +0.25%
large_transaction_processing::prepare 2.5±0.00ms 2.5±0.00ms 0.00%
large_transaction_processing::prepare_and_decompile 6.3±0.02ms 6.2±0.02ms -1.59%
large_transaction_processing::prepare_and_decompile_and_recompile 25.0±0.15ms 31.4±1.83ms +25.60%
metadata_validation::validate_urls 4.8±0.08µs 4.7±0.06µs -2.08%
precise_decimal::add/0 8.9±0.01ns 8.9±0.01ns 0.00%
precise_decimal::add/rust-native 10.7±0.01ns 10.7±0.01ns 0.00%
precise_decimal::add/wasmi 418.8±3.55ns 415.3±3.10ns -0.84%
precise_decimal::add/wasmi-call-native 3.9±0.01µs 3.9±0.01µs 0.00%
precise_decimal::div/0 296.8±2.11ns 286.5±0.23ns -3.47%
precise_decimal::from_string/0 202.0±0.14ns 201.9±0.11ns -0.05%
precise_decimal::mul/0 332.9±0.37ns 329.4±0.35ns -1.05%
precise_decimal::mul/rust-native 283.2±0.30ns 282.8±0.32ns -0.14%
precise_decimal::mul/wasmi 46.9±0.15µs 46.8±0.15µs -0.21%
precise_decimal::mul/wasmi-call-native 4.3±0.01µs 4.3±0.00µs 0.00%
precise_decimal::pow/0 1721.1±1.11ns 1742.9±3.27ns +1.27%
precise_decimal::pow/rust-native 1359.3±2.20ns 1354.3±2.01ns -0.37%
precise_decimal::pow/wasmi 223.1±0.93µs 228.4±0.41µs +2.38%
precise_decimal::pow/wasmi-call-native 7.5±0.03µs 7.5±0.01µs 0.00%
precise_decimal::root/0 58.1±0.02µs 58.9±0.03µs +1.38%
precise_decimal::sub/0 9.1±0.02ns 9.2±0.03ns +1.10%
precise_decimal::to_string/0 717.9±1.31ns 717.5±0.85ns -0.06%
schema::validate_payload 382.3±0.56µs 398.1±1.29µs +4.13%
transaction::radiswap 5.1±0.02ms 4.9±0.03ms -3.92%
transaction::transfer 1791.9±4.29µs 1843.3±2.61µs +2.87%
transaction_validation::validate_manifest 43.1±0.11µs 43.2±0.03µs +0.23%
transaction_validation::verify_bls_2KB 1046.3±43.67µs 1003.2±9.58µs -4.12%
transaction_validation::verify_bls_32B 1041.0±30.60µs 1002.4±8.77µs -3.71%
transaction_validation::verify_ecdsa 74.6±0.08µs 74.6±0.05µs 0.00%
transaction_validation::verify_ed25519 46.6±0.16µs 42.6±0.09µs -8.58%

@azizi-a azizi-a marked this pull request as ready for review January 6, 2025 16:56
@azizi-a azizi-a merged commit 0da03de into release/cuttlefish Jan 6, 2025
33 checks passed
@azizi-a azizi-a deleted the scrypto-dev-container-ci branch January 6, 2025 17:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@muzuke muzuke muzuke approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@azizi-a @muzuke