Skip to content

Update EIP-3026: fix input size specification in G2 multiexponentiation ABI #9711

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mdqst
Copy link
Contributor

@mdqst mdqst commented May 1, 2025

corrected an error in the ABI for G2 multiexponentiation section. It previously stated that the call expects 240*k bytes, but this contradicts the encoding sizes defined elsewhere (192 bytes for a G2 point + 64 bytes for a scalar = 256 bytes per slice). updated it to 256*k for consistency with the rest of the spec and the gas cost calculation logic.

also clarified the input length validation rule to explicitly require a multiple of 256.

@mdqst mdqst requested a review from eth-bot as a code owner May 1, 2025 11:59
@github-actions github-actions bot added c-update Modifies an existing proposal s-stagnant This EIP is Stagnant t-core labels May 1, 2025
@eth-bot
Copy link
Collaborator

eth-bot commented May 1, 2025

File EIPS/eip-3026.md

Requires 1 more reviewers from @hujw77, @iAmMichaelConnor, @yelhousni
Requires 1 more reviewers from @g11tech, @lightclient, @SamWilsn

@eth-bot eth-bot added the a-review Waiting on author to review label May 1, 2025
@eth-bot eth-bot changed the title docs: fix input size specification in G2 multiexponentiation ABI Update EIP-3026: fix input size specification in G2 multiexponentiation ABI May 1, 2025
Copy link

github-actions bot commented May 1, 2025

The commit 3b74d9b (as a parent of ba27bfb) contains errors.
Please inspect the Run Summary for details.

@github-actions github-actions bot added the w-ci Waiting on CI to pass label May 1, 2025
Copy link
Contributor

@g11tech g11tech left a comment

Choose a reason for hiding this comment

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

need approval from authors but this is anyway a stagnant EIP

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a-review Waiting on author to review c-update Modifies an existing proposal s-stagnant This EIP is Stagnant t-core w-ci Waiting on CI to pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants