Skip to content

refactor: add challenge randomness#36

Closed
silathdiir wants to merge 6 commits into
feat/refactor_noncefrom
refactor/challenge-api
Closed

refactor: add challenge randomness#36
silathdiir wants to merge 6 commits into
feat/refactor_noncefrom
refactor/challenge-api

Conversation

@silathdiir

@silathdiir silathdiir commented Apr 26, 2023

Copy link
Copy Markdown
Contributor

Reference zkevm-circuits PRs:

privacy-ethereum/zkevm-circuits#965
privacy-ethereum/zkevm-circuits#1020

Summary

  1. Add struct Challenges contains field mpt_word and function exprs and values.

  2. Use Challenges<Expression> in configure and Challenges<Value> in assign for randomness.

  3. Update related code of mpt_table and mpt_update.

@silathdiir silathdiir marked this pull request as draft April 26, 2023 10:04
@silathdiir silathdiir force-pushed the refactor/challenge-api branch 2 times, most recently from 816767c to 55d9764 Compare April 26, 2023 15:10
@silathdiir silathdiir force-pushed the refactor/challenge-api branch from 809652e to f8a3e55 Compare April 27, 2023 01:35
@silathdiir silathdiir changed the title [WIP] refactor: add Challenge API refactor: add challenge randomness Apr 27, 2023
@silathdiir silathdiir marked this pull request as ready for review April 27, 2023 01:41
Comment thread src/gadgets/mpt_update.rs Outdated
Comment thread src/gadgets/mpt_update.rs Outdated
silathdiir and others added 2 commits April 27, 2023 14:10
Co-authored-by: z2trillion <z2trillion@users.noreply.github.com>
Co-authored-by: z2trillion <z2trillion@users.noreply.github.com>
@z2trillion z2trillion changed the base branch from feat/refactor to feat/refactor_nonce May 23, 2023 01:09
@z2trillion

Copy link
Copy Markdown
Contributor

sorry, i ended up using #41 instead.

@z2trillion z2trillion closed this May 30, 2023
@z2trillion z2trillion deleted the refactor/challenge-api branch August 4, 2023 01:57
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.

2 participants