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

[DRAFT] eip-7251: Max Effective Balance Increase with Consolidations #13903

Draft
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

prestonvanloon
Copy link
Member

@prestonvanloon prestonvanloon commented Apr 22, 2024

What type of PR is this?

Feature

What does this PR do? Why is it needed?

Which issues(s) does this PR fix?

Tracking @ #13849

Other notes for review

Not ready for review yet

@prestonvanloon prestonvanloon added Pectra Electra electra hardfork labels Apr 22, 2024
terencechain added a commit that referenced this pull request Apr 22, 2024
@prestonvanloon prestonvanloon force-pushed the eip-7251 branch 4 times, most recently from 647fea2 to db50e07 Compare April 22, 2024 19:17
terencechain added a commit that referenced this pull request Apr 22, 2024
terencechain added a commit that referenced this pull request Apr 23, 2024
@prestonvanloon prestonvanloon force-pushed the eip-7251 branch 5 times, most recently from 3efc690 to 7a7d374 Compare April 24, 2024 20:15
terencechain added a commit that referenced this pull request Apr 24, 2024
@prestonvanloon prestonvanloon force-pushed the eip-7251 branch 2 times, most recently from 0d4ee28 to 6d43f1d Compare April 24, 2024 21:10
github-merge-queue bot pushed a commit that referenced this pull request Apr 24, 2024
* block protos

* proto and ssz

* stubs

* Enable Electra spec test

* Pull in EIP-7251 protobuf changes

From: #13903

* All EIP7549 containers are passing

* All EIP7251 containers passing

* including changes from eip7002

* Everything passing except for beacon state hash tree root

* fixing eletra state to use electra payload

* Fix minimal test. Skip beacon state test

* Perston's feedback

---------

Co-authored-by: rkapka <radoslaw.kapka@gmail.com>
Co-authored-by: james-prysm <james@prysmaticlabs.com>
@prestonvanloon prestonvanloon force-pushed the eip-7251 branch 8 times, most recently from 2dc3c74 to db0d175 Compare April 25, 2024 13:35
prestonvanloon pushed a commit that referenced this pull request Apr 25, 2024
* block protos

* proto and ssz

* stubs

* Enable Electra spec test

* Pull in EIP-7251 protobuf changes

From: #13903

* All EIP7549 containers are passing

* All EIP7251 containers passing

* including changes from eip7002

* Everything passing except for beacon state hash tree root

* fixing eletra state to use electra payload

* Fix minimal test. Skip beacon state test

* Perston's feedback

---------

Co-authored-by: rkapka <radoslaw.kapka@gmail.com>
Co-authored-by: james-prysm <james@prysmaticlabs.com>
prestonvanloon and others added 4 commits May 24, 2024 14:01
Spectests for process_effective_balance_updates

process_effective_balance_updates unit tests
Add has_compounding_withdrawal_credential helper

Progress

Tuesday night progress

Thursday night progress

Sunday afternoon progress

Tuesday progress

Monday night progress: tests are passing

Progress from last week, mostly on state

monday night progress: had to refactor a lot of tests to be blackbox tests

tuesday progress

wednesday progress

thursday progress

Friday progress

Monday progress

Fix up eip-7251 branch, fix issues, update to spec v1.5.0-alpha.0

More updates to spec 1.5.0-alpha.0, refactorings

Prepping for spectests

WIP: Electra beacon state HTR

Progress on custom HTR for beaconStateElectra

Rename eip7251 package to electra package

Working on beacon state changes and cleanup

Fix TestGetSpec

Fix ExpectedWithdrawals

Electra: full beacon config

Fixing conflicts

Resolve rebasing conflicts

Resolve rebasing conflicts

Update spectest to v1.5.0-alpha.1

Add PeerDAS config change

Fix API changes from beacon state PR
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Electra electra hardfork Pectra
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants