-
Notifications
You must be signed in to change notification settings - Fork 49
Speedup tests #1421
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
base: main
Are you sure you want to change the base?
Speedup tests #1421
Conversation
Benchmark results Main vs HEAD.Base
Head
Base
Head
Base
Head
Base
Head
Base
Head
Base
Head
|
Benchmarking resultsBenchmark for program
|
Command | Mean [s] | Min [s] | Max [s] | Relative |
---|---|---|---|---|
Cairo-vm (Rust, Cairo 1) |
10.910 ± 0.045 | 10.807 | 10.964 | 4.36 ± 0.04 |
cairo-native (embedded AOT) |
2.501 ± 0.019 | 2.484 | 2.542 | 1.00 |
cairo-native (embedded JIT using LLVM's ORC Engine) |
2.552 ± 0.016 | 2.528 | 2.587 | 1.02 ± 0.01 |
Benchmark for program dict_snapshot
Open benchmarks
Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
---|---|---|---|---|
Cairo-vm (Rust, Cairo 1) |
532.3 ± 8.5 | 524.5 | 547.5 | 1.00 |
cairo-native (embedded AOT) |
2187.7 ± 27.1 | 2150.0 | 2234.7 | 4.11 ± 0.08 |
cairo-native (embedded JIT using LLVM's ORC Engine) |
2279.7 ± 12.8 | 2262.8 | 2299.0 | 4.28 ± 0.07 |
Benchmark for program factorial_2M
Open benchmarks
Command | Mean [s] | Min [s] | Max [s] | Relative |
---|---|---|---|---|
Cairo-vm (Rust, Cairo 1) |
4.813 ± 0.039 | 4.772 | 4.890 | 1.85 ± 0.02 |
cairo-native (embedded AOT) |
2.602 ± 0.010 | 2.584 | 2.618 | 1.00 |
cairo-native (embedded JIT using LLVM's ORC Engine) |
2.648 ± 0.017 | 2.625 | 2.685 | 1.02 ± 0.01 |
Benchmark for program fib_2M
Open benchmarks
Command | Mean [s] | Min [s] | Max [s] | Relative |
---|---|---|---|---|
Cairo-vm (Rust, Cairo 1) |
4.740 ± 0.022 | 4.713 | 4.787 | 2.23 ± 0.01 |
cairo-native (embedded AOT) |
2.128 ± 0.010 | 2.111 | 2.142 | 1.00 |
cairo-native (embedded JIT using LLVM's ORC Engine) |
2.199 ± 0.034 | 2.163 | 2.252 | 1.03 ± 0.02 |
Benchmark for program linear_search
Open benchmarks
Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
---|---|---|---|---|
Cairo-vm (Rust, Cairo 1) |
573.2 ± 14.8 | 562.8 | 611.9 | 1.00 |
cairo-native (embedded AOT) |
2219.1 ± 19.1 | 2193.7 | 2243.5 | 3.87 ± 0.11 |
cairo-native (embedded JIT using LLVM's ORC Engine) |
2375.6 ± 31.8 | 2330.2 | 2434.6 | 4.14 ± 0.12 |
Benchmark for program logistic_map
Open benchmarks
Command | Mean [ms] | Min [ms] | Max [ms] | Relative |
---|---|---|---|---|
Cairo-vm (Rust, Cairo 1) |
381.6 ± 5.7 | 374.8 | 390.2 | 1.00 |
cairo-native (embedded AOT) |
2291.0 ± 19.7 | 2265.7 | 2326.6 | 6.00 ± 0.10 |
cairo-native (embedded JIT using LLVM's ORC Engine) |
2469.5 ± 17.6 | 2448.8 | 2499.1 | 6.47 ± 0.11 |
Co-authored-by: DiegoC <90105443+DiegoCivi@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Running make test
prints the following warning. Is the config file required?
warning: ignoring unknown configuration keys in config file /Users/julian/projects/cairo_native/.config/nextest.toml:
- profile.ci.debug
- profile.ci.inherits
- profile.ci.opt-level
Oh, I may have missed that message when I ran it. I'll check it. |
I ended up removing the config file. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Speedup tests
This PR adds cargo-nextest as the tool used to run tests. It also parallelizes how tests are run in
cairo-native-test
andscarb-native-test
which were run rather sequentially.Improvements:
make test-cairo
command.Introduces Breaking Changes?
No.
starknet-blocks.yml
workflow to use these PRs.These PRs should be merged after this one right away, in that order.
Checklist