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

Circuit for opcode SELFBALANCE #328

Merged

Conversation

scroll-dev
Copy link
Collaborator

Implement the SelfbalanceGadget for opcode SELFBALANCE.

Specs: privacy-scaling-explorations/zkevm-specs#113

Will update the bus mapping for SELFBALANCE once #292 is merged in upstream.

@github-actions github-actions bot added crate-bus-mapping Issues related to the bus-mapping workspace member T-opcode Type: opcode-related and focused PR/Issue labels Feb 10, 2022
@ChihChengLiang
Copy link
Collaborator

@scroll-dev Mason Liang Can we do a rebase for this?

@z2trillion z2trillion force-pushed the feat/opcode-selfbalance branch from 5906508 to 216d31a Compare February 14, 2022 15:35
@github-actions github-actions bot removed the crate-bus-mapping Issues related to the bus-mapping workspace member label Feb 14, 2022
@ed255 ed255 self-requested a review February 14, 2022 16:48
Copy link
Member

@ed255 ed255 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks, good work!

@ed255
Copy link
Member

ed255 commented Feb 18, 2022

@z2trillion #292 has been merged! Do you plan on updating this PR to implement the SELFBALANCE in the bus mapping? Or do you prefer doing that in another PR?
Either way the PR needs a rebase with main to be merged :)

@z2trillion z2trillion force-pushed the feat/opcode-selfbalance branch from cb07f2c to f3375e0 Compare February 23, 2022 20:03
@github-actions github-actions bot added the crate-bus-mapping Issues related to the bus-mapping workspace member label Feb 23, 2022
@z2trillion
Copy link
Collaborator

@ed255, I've rebased and added the bus mapping.

@ed255
Copy link
Member

ed255 commented Feb 24, 2022

@z2trillion There seems to be a linting error coming from clippy.

@z2trillion z2trillion force-pushed the feat/opcode-selfbalance branch from 522614a to 9441677 Compare February 24, 2022 16:25
@z2trillion
Copy link
Collaborator

@ed255 it should be fixed now.

@ed255 ed255 merged commit 1f4f1a8 into privacy-scaling-explorations:main Feb 24, 2022
@z2trillion z2trillion deleted the feat/opcode-selfbalance branch June 30, 2022 05:58
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 T-opcode Type: opcode-related and focused PR/Issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants