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

Improved encoding of objects passed to web3 #283

Merged
merged 6 commits into from
Sep 28, 2020
Merged

Conversation

dtebbs
Copy link
Contributor

@dtebbs dtebbs commented Sep 15, 2020

We encode verification keys, proofs and input values into evm words. This PR:

  • improves the interface to this functionality
  • makes the result more consistent over proofs / keys
  • supports converting elements from pairings other than alt-bn128.

(Depends on #280)

Note, there is still room for stronger types around the handling of keys and proofs in python.
This should be addressed in a future PR.

@dtebbs dtebbs changed the title WIP: Improved encoding of objects in evm words (depends on #280) Improved encoding of objects in evm words (depends on #280) Sep 16, 2020
client/zeth/core/zksnark.py Outdated Show resolved Hide resolved
client/tests/test_zksnark.py Show resolved Hide resolved
@dtebbs dtebbs force-pushed the precompiled-contracts branch 2 times, most recently from e8eabc4 to c07e2f1 Compare September 25, 2020 16:36
@AntoineRondelet AntoineRondelet changed the base branch from precompiled-contracts to develop September 28, 2020 08:23
@dtebbs dtebbs changed the title Improved encoding of objects in evm words (depends on #280) Improved encoding of objects passed to web3 (depends on #280) Sep 28, 2020
@dtebbs dtebbs changed the title Improved encoding of objects passed to web3 (depends on #280) Improved encoding of objects passed to web3 Sep 28, 2020
@AntoineRondelet
Copy link
Contributor

LGTM

@AntoineRondelet AntoineRondelet merged commit d668add into develop Sep 28, 2020
AntoineRondelet added a commit that referenced this pull request Oct 2, 2020
@AntoineRondelet AntoineRondelet deleted the evm-encoding branch October 14, 2020 10:28
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