This repository has been archived by the owner on Jul 5, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 858
[proof-chunk] Instantiate circuits with Chunk from bus-mapping #1690
Merged
hero78119
merged 74 commits into
privacy-scaling-explorations:proof-chunk
from
CeciliaZ030:bus-mapping-chunk-
Feb 1, 2024
Merged
[proof-chunk] Instantiate circuits with Chunk from bus-mapping #1690
hero78119
merged 74 commits into
privacy-scaling-explorations:proof-chunk
from
CeciliaZ030:bus-mapping-chunk-
Feb 1, 2024
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
new from block with chunk cecilia-wip
github-actions
bot
added
crate-bus-mapping
Issues related to the bus-mapping workspace member
crate-zkevm-circuits
Issues related to the zkevm-circuits workspace member
crate-integration-tests
Issues related to the integration-tests workspace member
labels
Nov 28, 2023
CeciliaZ030
changed the title
connect prev and cur fingerprint generated frm rw tables
Instantiate circuits with Chunk from bus-mapping
Nov 28, 2023
hero78119
changed the title
Instantiate circuits with Chunk from bus-mapping
[proof-chunk] Instantiate circuits with Chunk from bus-mapping
Nov 28, 2023
hero78119
reviewed
Nov 28, 2023
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.
Thanks for the PR!
First round quick passed and just few feedbacks
- it will be more clear to have PR description for what items has been covered in this PR along with TODO items.
- By quick overview I think current scope of PR is
- define chunk structure in bus-mapping
- convert chunk structure to chunk witness and bypass to circuit
I think current chunk structure might need to be further tuning when chunking flow are introduced, which seems to be in downstream work by @einar-taiko ? May I know whats plan to combining both works ?
This reverts commit 8ef6efe.
github-actions
bot
added
T-bench
Type: benchmark improvements
crate-circuit-benchmarks
Issues related to the circuit-benchmarks workspace member
labels
Dec 2, 2023
github-actions
bot
added
crate-mock
Issues related to the mock workspace member
crate-eth-types
Issues related to the eth-types workspace member
labels
Dec 3, 2023
github-actions
bot
removed
crate-mock
Issues related to the mock workspace member
crate-eth-types
Issues related to the eth-types workspace member
labels
Dec 3, 2023
Latest chunked scopeRed: directly chunked data are
Blue: chunking |
hero78119
reviewed
Jan 12, 2024
hero78119
reviewed
Jan 12, 2024
hero78119
reviewed
Jan 12, 2024
hero78119
reviewed
Jan 12, 2024
4 tasks
Reproduce heavy test evm circuit error
chores: fix bug
chores: bug fix
fix bug: add back missing constraints
Thanks for the contribution @CeciliaZ030 ⭐ |
hero78119
merged commit Feb 1, 2024
c017424
into
privacy-scaling-explorations:proof-chunk
13 checks passed
1 task
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Labels
crate-bus-mapping
Issues related to the bus-mapping workspace member
crate-circuit-benchmarks
Issues related to the circuit-benchmarks workspace member
crate-gadgets
Issues related to the gadgets workspace member
crate-integration-tests
Issues related to the integration-tests workspace member
crate-zkevm-circuits
Issues related to the zkevm-circuits workspace member
T-bench
Type: benchmark improvements
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Instantiate circuits with
Block
and specifiedChunk
from bus-mapping. RefactorCircuitInputBuilder
to generate given number ofChunk
s with fixed or dynamic params. Note that one instance of the circuit corresponds to one chunk instead of one block.Issue Link
#1696
Type of change
Contents
CircuitInputBuilder
with eitherFixedCParam
or just specify the total number of chunk and dynamically derive parameter after a dry run.handle_tx
split the chunk whenever the localrwc
exceeds the target amount, specifically by setting theEndChunk
and proceed to the nextBeginChunk
if needed.ChunkCtx
for each chunk at the end, which tracks the globalRw
range & localrwc
later used in the State circuits.builder.chunk_convert(idx)
, analogous tobuilder.block_convert()
.SubCircuit<F: Field>
trait includingnew_from_block(&block, &chunk)
andmin_num_rows_block(&block, &chunk)
.Workflow
For
CircuitInputBuilder
chunk is needed to build all circuits:For
CircuitTestBuilder
modifier is applied to both block and chunk, and you can run with fixed or dynamic params:Default
run()
still works since it runs one chunk internally.