-
Notifications
You must be signed in to change notification settings - Fork 998
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
release v1.3.0-alpha.2 #3161
Merged
release v1.3.0-alpha.2 #3161
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Future light client protocol extensions may include data from the block in addition to data from the state, e.g., `ExecutionPayloadHeader`. To prepare for this, also pass the block to the corresponding functions. In practice, blocks access is easier than historic state access, meaning there are no practical limitations due to this change.
Fork epoch override test decorators were still referring to `SHARDING`. Replace the implementation with one that needs less maintenance.
Fix type annotation in verify_aggregate_kzg_proof
Fix initialization of VERSIONED_HASH_VERSION_KZG
Replaces the "FAKE RLP HASH" in `ExecutionPayload.block_hash` of all generated tests with the correct value.
Additionally, it makes the Fiat-Shamir hashing logic more robust by making the challenges independent of each other. It also makes it more efficient to implement by moving both challenge computations to a single function needing a single transcript hash. Co-authored-by: George Kadianakis <desnacked@riseup.net> Co-authored-by: Dankrad Feist <mail@dankradfeist.de>
Fix EIP4844 `previous_version` field in tests
Co-authored-by: George Kadianakis <desnacked@riseup.net>
Add link to Capella's P2P doc
…eq-resp deprecate phase 0-only versions of beacon_blocks_by_{range,root}
Pass blocks to LC data creation functions
EIP4844: remove optimistic sync mentions
EIP4844: Update `MAX_BLOBS_PER_BLOCK` to 4
Fix typo in BlobsSidecarsByRange v1
bound the maximum number of validators considered for withdrawals per sweep
* Add @description decorator * Unify test case naming style * more clean ups * Altair tests cleanup * Clean up Altair and Bellatrix `process_deposit` tests * Clean up Bellatrix tests * Clean up Capella tests * PR feedback from @ralexstokes * Add comments on the deposit fork version tests * Remove `test_incorrect_sig_other_version` since it is duplicate to `test_ineffective_deposit_with_bad_fork_version` * Add `test_ineffective_deposit_with_current_fork_version`
Co-authored-by: Hsiao-Wei Wang <hsiaowei.eth@gmail.com>
proper EL block hash implementation in tests
cleanup explicit fork enumerations in tests
Fix some nits in eip4844 specs
Add tests to test sync aggregate's order of balance operation
Update G2 trusted setup length to 65
EIP4844: Clarify that z should not be in the domain
potuz
approved these changes
Dec 14, 2022
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Release this :)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
DO NOT MERGE
todo: