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

test: enable benchmarking with rapidsnark #357

Merged
merged 10 commits into from
Apr 4, 2024

Conversation

chokobole
Copy link
Contributor

Description

This PR includes benchmarks comparing Tachyon and Rapidsnark, revealing that ours runs 10x faster than Rapidsnark!

@chokobole chokobole force-pushed the test/enable-benchmark-with-rapidsnark branch 2 times, most recently from bce2229 to 42711f2 Compare March 27, 2024 09:47
@chokobole chokobole marked this pull request as ready for review March 27, 2024 09:47
@chokobole chokobole force-pushed the test/enable-benchmark-with-rapidsnark branch from 42711f2 to 4bb1933 Compare March 27, 2024 14:03
Copy link
Contributor

@Insun35 Insun35 left a comment

Choose a reason for hiding this comment

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

Please specify the scope and fix typo on your commit message(a557c8c)
build: resolve ambiguous funciton call issue -> build(math): resolve ambiguous function call issue

@ashjeong ashjeong changed the title test: enable benchmark with rapidsnark test: enable benchmarking with rapidsnark Mar 28, 2024
@chokobole chokobole force-pushed the test/enable-benchmark-with-rapidsnark branch from 4bb1933 to ca7556c Compare March 28, 2024 15:16
Copy link
Contributor

@ashjeong ashjeong left a comment

Choose a reason for hiding this comment

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

1bbbdd7
test(circom): enable benchmark with rapidsnark
->
test(circom): enable benchmarking with rapidsnark

vendors/circom/README.md Outdated Show resolved Hide resolved
vendors/circom/README.md Outdated Show resolved Hide resolved
vendors/circom/README.md Outdated Show resolved Hide resolved
vendors/circom/README.md Outdated Show resolved Hide resolved
vendors/circom/README.md Outdated Show resolved Hide resolved
vendors/circom/benchmark/README.md Outdated Show resolved Hide resolved
vendors/circom/benchmark/README.md Outdated Show resolved Hide resolved
vendors/circom/README.md Outdated Show resolved Hide resolved
vendors/circom/README.md Outdated Show resolved Hide resolved
vendors/circom/README.md Outdated Show resolved Hide resolved
Copy link
Contributor

@Insun35 Insun35 left a comment

Choose a reason for hiding this comment

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

Else, LGTM

vendors/circom/benchmark/circom_benchmark.cc Outdated Show resolved Hide resolved
@chokobole chokobole force-pushed the test/enable-benchmark-with-rapidsnark branch 2 times, most recently from 7b31fef to 818dd75 Compare April 2, 2024 08:37
@chokobole chokobole force-pushed the test/enable-benchmark-with-rapidsnark branch from 818dd75 to a100630 Compare April 2, 2024 12:32
Copy link
Contributor

@TomTaehoonKim TomTaehoonKim left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@dongchangYoo dongchangYoo left a comment

Choose a reason for hiding this comment

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

LGTM

In alignment with our documentation standards, this commit updates instances of
`Benchmark` to `Benchmarking` in files where the documentation contains
more than just the result.
@chokobole chokobole force-pushed the test/enable-benchmark-with-rapidsnark branch from a100630 to 4ba3678 Compare April 3, 2024 08:59
Copy link
Contributor

@ashjeong ashjeong left a comment

Choose a reason for hiding this comment

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

LGTM

@chokobole chokobole merged commit 965c071 into main Apr 4, 2024
3 checks passed
@chokobole chokobole deleted the test/enable-benchmark-with-rapidsnark branch April 4, 2024 04:56
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

5 participants