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

Added templ_subdir flag to cargo risczero new #517

Merged
merged 3 commits into from
Apr 18, 2023
Merged

Conversation

mothran
Copy link
Contributor

@mothran mothran commented Apr 18, 2023

Provides the optional ability to select from multiple template locations within a folder|repo at --template

risc0/cargo-risczero/src/commands/new.rs Outdated Show resolved Hide resolved
Co-authored-by: Tim Zerrell <tim.zerrell@risczero.com>
@github-actions
Copy link

Benchmark for Linux-cuda 28b3876

Click to hide benchmark
Test Base PR %
fib/100/proof 867.2±40.85ms 844.5±47.69ms -2.62%
fib/100/run 333.6±23.03ms 320.7±22.99ms -3.87%
fib/200/proof 912.7±45.97ms 852.7±64.99ms -6.57%
fib/200/run 343.6±21.43ms 323.0±16.55ms -6.00%

Benchmark for Linux-default 28b3876

Click to hide benchmark
Test Base PR %
fib/100/proof 2.3±0.02s 2.3±0.02s 0.00%
fib/100/run 323.1±5.13ms 321.3±7.37ms -0.56%
fib/200/proof 2.3±0.02s 2.3±0.01s 0.00%
fib/200/run 327.1±4.29ms 326.9±7.02ms -0.06%

Benchmark for macOS-default 28b3876

Click to hide benchmark
Test Base PR %
fib/100/proof 1604.6±21.89ms 1598.3±7.15ms -0.39%
fib/100/run 111.8±0.76ms 111.4±0.61ms -0.36%
fib/200/proof 1599.1±9.86ms 1595.8±8.46ms -0.21%
fib/200/run 113.5±0.76ms 113.0±0.52ms -0.44%

Benchmark for macOS-metal 28b3876

Click to hide benchmark
Test Base PR %
fib/100/proof 465.6±3.64ms 463.9±4.28ms -0.37%
fib/100/run 114.1±1.79ms 113.2±0.43ms -0.79%
fib/200/proof 469.3±6.89ms 467.5±8.14ms -0.38%
fib/200/run 115.5±0.51ms 115.5±0.20ms 0.00%

@github-actions
Copy link

Benchmark for Linux-cuda 8c66e38

Click to hide benchmark
Test Base PR %
fib/100/proof 833.1±26.84ms 809.3±39.99ms -2.86%
fib/100/run 340.2±17.77ms 336.3±20.04ms -1.15%
fib/200/proof 841.8±44.42ms 822.7±32.80ms -2.27%
fib/200/run 344.9±12.29ms 343.7±28.14ms -0.35%

Benchmark for Linux-default 8c66e38

Click to hide benchmark
Test Base PR %
fib/100/proof 2.3±0.02s 2.3±0.02s 0.00%
fib/100/run 336.7±0.92ms 323.8±6.03ms -3.83%
fib/200/proof 2.3±0.01s 2.3±0.01s 0.00%
fib/200/run 344.8±4.42ms 330.4±3.66ms -4.18%

Benchmark for macOS-default 8c66e38

Click to hide benchmark
Test Base PR %
fib/100/proof 1596.7±7.31ms 1594.5±10.91ms -0.14%
fib/100/run 112.5±0.40ms 111.1±0.68ms -1.24%
fib/200/proof 1600.6±8.49ms 1598.3±11.70ms -0.14%
fib/200/run 114.4±0.60ms 112.8±0.48ms -1.40%

Benchmark for macOS-metal 8c66e38

Click to hide benchmark
Test Base PR %
fib/100/proof 465.3±4.02ms 464.2±1.79ms -0.24%
fib/100/run 113.6±1.05ms 113.4±0.22ms -0.18%
fib/200/proof 469.9±4.13ms 469.1±7.12ms -0.17%
fib/200/run 115.5±0.31ms 115.2±0.44ms -0.26%

@mothran mothran merged commit 9092149 into main Apr 18, 2023
10 checks passed
@mothran mothran deleted the parker/new-subdir branch April 18, 2023 20:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants