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

v0.8.1: (1) aggregate 15 chunks (2) parallel assignment #262

Merged
merged 11 commits into from
Sep 7, 2023
Merged

Conversation

silathdiir
Copy link
Contributor

@silathdiir silathdiir commented Sep 6, 2023

Related zkevm-circuits PR scroll-tech/zkevm-circuits#803

Summary

  • prover/test_data/full_proof_1.json is chunk-179001 to 179015 of sepolia.
  • need to help check changes of layer3.config and layer4.config. @zhenfeizhang
  • layer-4 could not work with degree=26 and num_advice=1 (I fixed to 2).

batch-prove.log.tar.gz

Test

batch-prove test could pass.

Screen Shot 2023-09-06 at 3 11 15 PM

@silathdiir silathdiir requested a review from Velaciela September 6, 2023 08:00
This reverts commit aa997ef.
@zhenfeizhang
Copy link
Contributor

scroll-tech/zkevm-circuits#803 MAX_SNARK=15 passes with the following config

layer 3
{
  "strategy": "Simple",
  "degree": 20,
  "num_advice": [
    59
  ],
  "num_lookup_advice": [
    7
  ],
  "num_fixed": 2,
  "lookup_bits": 18,
  "limb_bits": 88,
  "num_limbs": 3
}
layer 4
{
  "strategy": "Simple",
  "degree": 26,
  "num_advice": [
    1
  ],
  "num_lookup_advice": [
    1
  ],
  "num_fixed": 1,
  "lookup_bits": 20,
  "limb_bits": 88,
  "num_limbs": 3
}

@lispc lispc changed the title [don't-merge] test max-snark v0.8.1: (1) aggregate 15 chunks (2) parallel assignment Sep 6, 2023
@lispc lispc merged commit b6d98c3 into main Sep 7, 2023
5 checks passed
@lispc lispc deleted the test-max-snark branch September 7, 2023 01:58
@silathdiir silathdiir restored the test-max-snark branch September 13, 2023 12:13
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.

3 participants