-
Notifications
You must be signed in to change notification settings - Fork 147
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
Search space restriction #1125
Search space restriction #1125
Conversation
.../concrete-optimizer/concrete-optimizer/src/optimization/dag/multi_parameters/optimize/mod.rs
Show resolved
Hide resolved
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. Let's see with real implem.
6db7910
to
266da78
Compare
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.
Benchmark
Benchmark suite | Current: f70a18b | Previous: 09894b6 | Ratio |
---|---|---|---|
v0 PBS table generation |
53763602 ns/iter (± 521216 ) |
53863508 ns/iter (± 965902 ) |
1.00 |
v0 PBS simulate dag table generation |
38420347 ns/iter (± 525977 ) |
38423465 ns/iter (± 390678 ) |
1.00 |
v0 WoP-PBS table generation |
52198086 ns/iter (± 282530 ) |
52551878 ns/iter (± 494024 ) |
0.99 |
This comment was automatically generated by workflow using github-action-benchmark.
8796b5d
to
d540e48
Compare
d540e48
to
6e35b76
Compare
fca910d
to
0fccfbf
Compare
compilers/concrete-compiler/compiler/lib/Bindings/Python/CompilerAPIModule.cpp
Outdated
Show resolved
Hide resolved
compilers/concrete-optimizer/concrete-optimizer-cpp/src/concrete-optimizer.rs
Show resolved
Hide resolved
.../concrete-optimizer/concrete-optimizer/src/optimization/dag/multi_parameters/optimize/mod.rs
Show resolved
Hide resolved
cc @jdmarsh the API you do will change with this PR |
3849f7c
to
09894b6
Compare
09894b6
to
f70a18b
Compare
No description provided.