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

[wip] preallocate msm #1038

Closed
wants to merge 4 commits into from
Closed

[wip] preallocate msm #1038

wants to merge 4 commits into from

Conversation

winston-h-zhang
Copy link
Member

No description provided.

@winston-h-zhang
Copy link
Member Author

!gpu-benchmark

@winston-h-zhang
Copy link
Member Author

!gpu-benchmark

Copy link
Contributor

Benchmark for 1751b93

Click to view benchmark
Test Base PR %
LEM Fibonacci Prove - rc = 100/fib/num-100 1727.7±2.03ms 2.3±0.00s +33.12%
LEM Fibonacci Prove - rc = 100/fib/num-200 3.3±0.00s 4.4±0.01s +33.33%
LEM Fibonacci Prove - rc = 600/fib/num-100 1962.2±8.48ms 2.3±0.01s +17.22%
LEM Fibonacci Prove - rc = 600/fib/num-200 3.3±0.01s 4.3±0.01s +30.30%

Copy link
Contributor

Benchmark for 1751b93

Click to view benchmark
Test Base PR %
LEM Fibonacci Prove - rc = 100/fib/num-100 1730.7±2.37ms 1718.1±1.98ms -0.73%
LEM Fibonacci Prove - rc = 100/fib/num-200 3.3±0.00s 3.3±0.00s 0.00%
LEM Fibonacci Prove - rc = 600/fib/num-100 1956.3±7.14ms 2.0±0.01s +2.23%
LEM Fibonacci Prove - rc = 600/fib/num-200 3.3±0.01s 3.4±0.01s +3.03%

@huitseeker
Copy link
Member

Is this still relevant? if not, should we close this?

@huitseeker
Copy link
Member

Closing in favor of #1107

@huitseeker huitseeker closed this Feb 19, 2024
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.

2 participants