Skip to content
This repository was archived by the owner on Jul 5, 2024. It is now read-only.

[presentation] 4 lookup cols#681

Closed
ChihChengLiang wants to merge 1 commit into
mainfrom
four-cols
Closed

[presentation] 4 lookup cols#681
ChihChengLiang wants to merge 1 commit into
mainfrom
four-cols

Conversation

@ChihChengLiang
Copy link
Copy Markdown
Collaborator

Try 4 lookup columns per @han0110's comment #596 (review).

We now have one extra unused advice column. If we modify the generic gate in some way we can use it to reduce some rows.

Num advice columns: 4
Num fixed columns: 7
Num instance columns: 0
Num lookups: 1
Num gates: 1
Max advice rows 64794 - add advice
num polys: 1
max_gate_degree_poly: 3 - 
Circuit layout: Click to expand!

keccak-f

@github-actions github-actions Bot added the crate-keccak Issues related to the keccak workspace member label Aug 12, 2022
@ChihChengLiang ChihChengLiang added the benchmarks: Keccak Triggers a Keccak round benchmark label Aug 12, 2022
@ChihChengLiang
Copy link
Copy Markdown
Collaborator Author

close this presentation pr

@ChihChengLiang ChihChengLiang deleted the four-cols branch May 17, 2023 08:14
lispc pushed a commit that referenced this pull request Aug 28, 2023
Co-authored-by: naure <naure@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

benchmarks: Keccak Triggers a Keccak round benchmark crate-keccak Issues related to the keccak workspace member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant