3d9c87b27c
Move to misc helpers
2023-03-30 09:44:34 +09:00
f696b30608
Address PR review
2023-03-30 09:11:36 +09:00
c7029ce19e
Rename get_epoch_boundary_block to get_ancestor_at_epoch_boundary
2023-03-29 12:40:58 +11:00
314b040fff
Reduce line len
2023-03-28 17:45:52 +09:00
cd7783e59d
EIP-6110: Fix `compute_fork_version` and add light client specs
2023-03-28 15:08:03 +08:00
c1273bbfa4
Add epoch > REUSE_VALIDATOR_INDEX_DELAY condition
2023-03-28 15:46:01 +09:00
ee3e1ac63e
Apply review comments
2023-03-28 15:34:07 +09:00
8acc31adce
Fix the default testing genesis by setting `deposit_receipts_start_index` and fork versions
2023-03-28 14:23:23 +08:00
ddbd82e1be
Add toc
2023-03-28 15:51:34 +11:00
201f113b50
Introduce get_epoch_boundary_block
2023-03-28 15:43:48 +11:00
f9b359be09
Reuse indexes with full sweep
2023-03-28 12:44:21 +09:00
3115d1140b
Fix: typos
2023-03-25 13:14:57 +01:00
a9abfeb217
Denbeb -> Deneb
2023-03-23 17:21:41 -07:00
334601c032
Merge pull request #3299 from asn-d6/inv_no_inputs
...
Reject zero inputs in bls_modular_inverse()
2023-03-20 14:18:58 +08:00
18eb1fa334
Set `CAPELLA_FORK_EPOCH` to `194048`
2023-03-20 13:48:49 +08:00
0e0403d0e2
Reject zero inputs in bls_modular_inverse()
2023-03-17 11:50:53 +02:00
de57f2ac91
Merge pull request #3284 from etan-status/lc-forkversion
...
Check correct fork version in LC sync protocol
2023-03-16 09:08:01 +08:00
985fcc2525
Merge pull request #3283 from ethereum/deneb-crypto-coverage
...
Deneb crypto helpers test coverage
2023-03-15 11:28:43 -06:00
5977f36fef
minor nits from code review
2023-03-15 10:54:49 -06:00
41f4472ad5
Merge pull request #3291 from kevaundray/patch-10
...
Deneb : Change description for blob
2023-03-15 10:51:39 -06:00
a0eb23f108
Merge pull request #3290 from ethereum/fork-choice-upgrade
...
Fork choice upgrade
2023-03-15 10:51:09 -06:00
8fd22ab504
code review
2023-03-15 15:25:09 +00:00
3141806100
Merge branch 'dev' into deneb-crypto-coverage
2023-03-14 22:45:06 +00:00
3e281e7457
Alternative for linter
2023-03-14 22:22:23 +00:00
2d4bfabceb
Correct signature of get_blobs_and_kzg_commitments
2023-03-14 22:16:41 +00:00
ff7a6c5d0e
Add description when to use verify_sidecar_signature
2023-03-14 22:14:31 +00:00
1a2967a542
Apply suggestions from @djrtwo
2023-03-14 14:49:41 -07:00
637ef34133
fix : change description for blob
2023-03-14 21:00:20 +00:00
3de96f7a19
Apply suggestions from code review
...
Co-authored-by: Danny Ryan <dannyjryan@gmail.com>
2023-03-14 13:54:57 -07:00
f7837e84e5
Merge pull request #3275 from tbenr/patch-1
...
Add `blob_sidecar` gossip rule for parent slot
2023-03-14 12:15:38 -06:00
b53392b463
Merge pull request #3177 from mkalinin/deposits
...
In-protocol deposits flow (no queue approach)
2023-03-14 11:33:36 -06:00
0ae18d86e3
Update specs/_features/eip6110/validator.md
...
Co-authored-by: Hsiao-Wei Wang <hsiaowei.eth@gmail.com>
2023-03-14 11:22:12 -06:00
23c5b7c66e
fix typo
2023-03-13 13:48:34 -07:00
e9551715d5
large fork choice upgrade
2023-03-13 11:52:43 -07:00
df4ba47e63
Update beacon-chain.md
2023-03-13 08:51:02 +08:00
cc284b2b60
Toc
2023-03-12 23:11:40 +00:00
96ad61bcec
Add tests for blob sidecar signature
2023-03-12 23:05:01 +00:00
29b5309f7d
Add proofs to validate_blobs_and_kzg_commitments
2023-03-11 23:02:35 +00:00
5e74c51411
Tests for validate_blobs_and_kzg_commitments and fix
2023-03-11 22:34:41 +00:00
6b69450992
fix typo in type of KZG_SETUP_LAGRANGE
2023-03-11 15:44:42 +00:00
47f078fc1f
Move change to deneb
2023-03-11 20:28:49 +08:00
1c35eb1c33
Lock voluntary_exit domain on capella
2023-03-10 14:37:18 +08:00
21cf61ea8f
De-duplicate range references in BlobSidecarsByRange
2023-03-10 11:10:33 +08:00
0da79bdbfd
Provide validator guide for EIP6110
2023-03-09 21:05:07 +06:00
7f74a08a6c
Fix trailing whitespace
2023-03-09 11:07:01 +00:00
54c7df5bbf
Fix lint
2023-03-08 15:42:56 +01:00
43e714e60f
Check correct fork version in LC sync protocol
...
- Sync committee is determined by signature_slot
- Signature fork version is determined by max(signature_slot, 1) - 1
- Attested block fork version can be anything < signature_slot
Old logic incorrectly derived signature fork version from signature_slot
and did not subtract a slot. Extended tests to check this edge case.
2023-03-08 15:34:56 +01:00
ccfe576dcc
Add KZG tests for input length inputs ( #3282 )
2023-03-07 21:56:55 +00:00
15033d28b9
Modify compute_[blob_]kzg_proof to remove superfluous computations ( #3280 )
...
Add parameter `commitment` to `compute_blob_kzg_proof` and output `y` to `compute_kzg_proof`
2023-03-07 17:50:56 +00:00
1b4840c967
Fix comment for `evaluate_polynomial_in_evaluation_form` to reflect that it can now also be used in the domain
2023-03-04 19:20:01 +00:00
86fb82b221
Test generators for kzg-4844 libraries ( #3274 )
...
Arkworks integration and test generators for kzg-4844 libraries
2023-03-02 20:49:10 +00:00
00557c5649
Remove unnecessary eth1_deposit_index bump
2023-03-02 17:31:12 +06:00
13f3654296
Apply suggestions from @djrtwo
2023-03-02 17:29:22 +06:00
c445fa9b37
Apply suggestions from code review
...
Co-authored-by: Danny Ryan <dannyjryan@gmail.com>
2023-03-02 15:50:08 +06:00
3259922a9e
change usage of MIN_EPOCHS_FOR_BLOBS_SIDECARS_REQUESTS
2023-03-01 17:10:58 +00:00
7538e9a4f3
Merge branch 'dev' into 4844-remove-empty-container
2023-03-01 08:58:42 -07:00
a236770b07
EIP4844: Use `MAX_REQUEST_BLOB_SIDECARS`
2023-02-28 15:17:40 -08:00
4a59bcfaa9
Merge branch 'dev' into deposits
2023-03-01 00:10:30 +08:00
6baa9531a4
Merge pull request #3250 from benjaminion/get-weight
...
Change get_latest_attesting_balance() to get_weight()
2023-02-28 21:29:22 +08:00
7bb65f88d9
Cosmetic fix
2023-02-28 17:18:07 +06:00
fae77eb53d
Apply @hwwhww suggestions
2023-02-28 16:36:46 +06:00
de5be63399
Apply suggestions from code review
...
Co-authored-by: Hsiao-Wei Wang <hsiaowei.eth@gmail.com>
2023-02-28 16:14:47 +06:00
0879c46a34
Add `blob_sidecar` gossip rule for parent slot
...
Similarly to the check we do on Block gossip, we should check slot consistency with the parent block, so we can independently reject wrong block and blobb_sidecar when the rule is violated.
2023-02-27 20:12:31 +01:00
429dc5daba
Merge pull request #3271 from henridf/patch-2
...
Update fork-choice.md
2023-02-27 09:20:25 -07:00
5474b34466
Merge branch 'henridf-max-request-blobs' into dev
2023-02-27 09:19:23 -07:00
d3d7ad6e97
Merge pull request #3270 from ethereum/deneb-fork
...
Remove the outdated description in Deneb fork
2023-02-27 09:15:24 -07:00
a26556147c
Merge pull request #3268 from henridf/block-blobc
...
Minor typo/pasta fixes in blob_sidecar_{index} validation
2023-02-27 09:12:40 -07:00
136c78ddc7
Update fork-choice.md
...
Fix outdated (likely a Bellatrix cut-paste) description of change.
2023-02-24 14:07:16 +01:00
9d690a4cb2
Fix typo
2023-02-24 17:58:10 +08:00
703fdfc7c7
Fix linter
2023-02-23 21:31:19 +06:00
7d6831ec86
Fix initialize_beacon_state_from_eth1 definition
2023-02-23 21:23:52 +06:00
e7035dacf5
Remove the outdated statement
2023-02-23 22:46:55 +08:00
a1daac098c
Make EIP-6110 executable and fix linter errors
2023-02-23 22:34:32 +08:00
b22c89244a
Cosmetic renaming
2023-02-23 14:09:01 +06:00
23c10cfd7f
Remove state.deposit_receipt_next_index variable
2023-02-23 13:53:15 +06:00
970da9efd2
Clean up max request blobs constants
...
The spec currently defines `MAX_REQUEST_BLOB_SIDECARS` as the "maximum
number of blob sidecars in a single request", but then later in the
RPC description defines the max is `MAX_REQUEST_BLOB_SIDECARS *
MAX_BLOBS_PER_BLOCK`.
Clean this up by defining `MAX_REQUEST_BLOB_SIDECARS` to be the actual max.
2023-02-22 17:20:44 +01:00
d8111d7d3b
Refer to "sidecar" consistently
2023-02-22 16:51:56 +01:00
837233a1be
Fix reference to block->sidecar
...
(This was probably a cut-n-paste from block validation)
2023-02-22 16:50:56 +01:00
08c7287387
Get rid of pending_deposits queue
2023-02-22 18:33:05 +06:00
9391f3ccfc
fix MAX_REQUEST_BLOBS_SIDECARS typo
2023-02-21 21:48:21 -06:00
d5c7474d4d
Move EIP6110 to features
2023-02-21 17:31:27 +06:00
e64607f7db
Merge branch 'dev' into deposits
2023-02-21 17:26:30 +06:00
95401cf6e4
Clarify context bytes in the RPC methods in 4844
2023-02-21 01:10:40 +08:00
ad58bfc304
Merge pull request #3244 from arnetheduck/back-to-the-decoupled-future
...
EIP-4844: Free the blobs
2023-02-20 09:40:56 -07:00
ac0ec660d3
add parent validation requirement
...
sama as block
2023-02-20 16:35:52 +01:00
7fd36f7ce0
Merge pull request #3262 from kevaundray/patch-7
...
EIP4844: Remove repeated computation in polynomial-commitments.md
2023-02-20 22:45:31 +08:00
4e2a9920f1
Update specs/deneb/p2p-interface.md
...
Co-authored-by: g11tech <develop@g11tech.io>
2023-02-20 12:15:53 +01:00
83cf02f668
Remove repeated computation
2023-02-20 10:57:39 +00:00
0632a5a32c
Update specs/deneb/polynomial-commitments.md
...
Co-authored-by: Hsiao-Wei Wang <hsiaowei.eth@gmail.com>
2023-02-20 10:54:16 +00:00
54d2559eb5
remove producer reorg on multi-blob
...
* also, use root/index for uniqueness
2023-02-18 17:45:16 +01:00
c1a2962b31
Update polynomial-commitments.md
2023-02-18 15:09:43 +00:00
9dd7d2ba2f
fix Blob pluralization in a few places
2023-02-17 11:59:56 -07:00
a562710fe6
Fix `compute_quotient_eval_within_domain` overflow
2023-02-17 01:22:11 +08:00
a7e45db9ac
Fix `verify_kzg_proof_batch` and the tests
2023-02-16 22:14:14 +08:00
f23ed0cdbc
Make linter happy
2023-02-16 21:11:18 +08:00
c2315c90a3
Merge branch 'dev' into pr3244
2023-02-16 21:07:36 +08:00
ab11a014ee
Merge pull request #3248 from ethereum/mv-features
...
Move experimental features to `specs/_features` folder
2023-02-16 17:03:19 +08:00
5fe857b209
fixes
2023-02-16 09:20:40 +01:00
24a19bb886
fixes
...
* fight the test suite
* clarify who orphans the block
* must supply all blobs of a block in range request
2023-02-16 09:12:34 +01:00
639ff9b2b0
Update specs/deneb/p2p-interface.md
...
Co-authored-by: Jimmy Chen <jchen.tc@gmail.com>
2023-02-16 08:30:40 +01:00
c39fda19c6
Apply suggestions from code review
...
Co-authored-by: Danny Ryan <dannyjryan@gmail.com>
Co-authored-by: Jimmy Chen <jchen.tc@gmail.com>
2023-02-16 08:18:52 +01:00
078d62e6ff
Simplify compute_challenge
2023-02-15 19:48:58 +00:00
48e7be7dd0
Fix doctoc
2023-02-15 18:23:04 +00:00
adb5134266
Merge branch 'dev' into kzg_multi_verify
2023-02-15 18:10:39 +00:00
7637158a2f
Change get_latest_attesting_balances() to get_weight()
2023-02-15 11:39:33 +00:00
f0dc126602
doctoc
2023-02-15 09:10:31 +01:00
a5f61fc173
correct function
2023-02-15 08:57:23 +01:00
da34af97d4
simplify blob verification, range request
...
* validate blobs using raw types
* remove `BlobSidecars` and send flattened list of `BlobSidecar`
instances instead
2023-02-15 08:51:57 +01:00
3a37c3c497
Allow clients to orphan blocks from spammy proposers
...
Proposers that spam the blob topic with multiple blob versions, some of
which are invalid, MAY see their block orphaned.
2023-02-15 08:00:39 +01:00
4086a09d0f
multi -> batch
2023-02-14 21:21:46 +00:00
c3cb7fa773
Comment on compute_challenge, assert on verify_blob_kzg_proof_multi
2023-02-14 21:12:02 +00:00
d8509e42c6
Update specs/deneb/polynomial-commitments.md
...
Co-authored-by: George Kadianakis <desnacked@riseup.net>
2023-02-14 20:59:41 +00:00
aafbd45a19
Update specs/deneb/polynomial-commitments.md
...
Co-authored-by: George Kadianakis <desnacked@riseup.net>
2023-02-14 20:59:24 +00:00
3a6fccd389
Remove double hashing
2023-02-14 20:17:25 +00:00
855cf062f0
Remove additional function
2023-02-14 20:07:34 +00:00
c49a2c2855
Update specs/deneb/polynomial-commitments.md
...
Co-authored-by: George Kadianakis <desnacked@riseup.net>
2023-02-14 20:00:58 +00:00
db5a168f3b
Move experimental features to `specs/_features` folder
2023-02-14 23:41:40 +08:00
89f2eae318
Merge pull request #3218 from protolambda/excess-data-gas-field-position
...
eip4844: move excess data gas field to end of execution payload for merkle proof path compat
2023-02-14 07:50:25 -07:00
58207c1c05
Upper limit on indices
2023-02-14 14:18:29 +01:00
fc4e1a9acf
EIP4844: compute_kzg_proof() can now create proofs within the domain ( #3243 )
...
This will be used by optimistic rollups to create proofs about past data
2023-02-14 14:50:44 +02:00
e6b8324e25
sidecar domain
2023-02-14 13:39:59 +01:00
c8719f8524
Apply suggestions from code review
...
Co-authored-by: Danny Ryan <dannyjryan@gmail.com>
2023-02-14 13:32:18 +01:00
86d955ab7f
Call compute_challenge with polynomial as argument
2023-02-13 17:17:41 +00:00
7f1748b3c8
Change blob verification fiat-shamir to single blob
2023-02-13 17:16:15 +00:00
b76ea49fec
Add KZG multi verify function
2023-02-13 17:12:51 +00:00
f91b9863ce
Simplify commitee weight computation
2023-02-13 17:08:52 +00:00
470c1b14b3
fix references to eip4844
2023-02-13 17:08:51 +00:00
901303f14f
Replaced EIP4844 references with Deneb
...
Fixes #3207
2023-02-13 17:08:51 +00:00
7b642a2884
compute_challenge takes blobs + linter
2023-02-13 14:57:04 +00:00
f36925ea96
Merge branch 'dev' into kzg_multi_verify
2023-02-13 14:38:19 +00:00
fc10714f42
Call compute_challenge with polynomial as argument
2023-02-13 14:32:50 +00:00
86962b9437
Simplify commitee weight computation
2023-02-10 11:43:38 -03:00
8bc19d99ae
fixes
...
* expand sidecar gossip conditions
* editing
* add spec text for `BlobSidecar` signatures
2023-02-10 11:16:51 +01:00
20278d4841
Merge remote-tracking branch 'origin/dev' into back-to-the-decoupled-future
...
Deneb merge
2023-02-10 10:44:31 +01:00
ffc78e9928
fixes
...
* separate constant for blob requests
* pedantry
2023-02-10 10:40:43 +01:00
368e70d9be
Remove sending empty blobs sidecar responses
2023-02-10 10:45:07 +08:00
7b5acbfd21
Fix Capella fork test assertions
2023-02-10 00:30:00 +08:00
f6b8827eca
eip4844: move excess data gas field to end of execution payload for merkle proof path compat
2023-02-10 00:29:58 +08:00
6cc5b0fc8b
Merge branch 'dev' into capella-fc-cleanup
2023-02-10 00:24:10 +08:00
078e1cc871
fix references to eip4844
2023-02-08 09:22:28 +10:00
deb82e2f26
fix member
2023-02-07 11:23:18 +01:00
1e07685f74
doctoc
2023-02-07 11:14:59 +01:00
65d3c6aeb6
Free the blobs
...
This PR reintroduces and further decouples blocks and blobs in EIP-4844,
so as to improve network and processing performance.
Block and blob processing, for the purpose of gossip validation, are
independent: they can both be propagated and gossip-validated
in parallel - the decoupled design allows 4 important optimizations
(or, if you are so inclined, removes 4 unnecessary pessimizations):
* Blocks and blobs travel on independent meshes allowing for better
parallelization and utilization of high-bandwidth peers
* Re-broadcasting after validation can start earlier allowing more
efficient use of upload bandwidth - blocks for example can be
rebroadcast to peers while blobs are still being downloaded
* bandwidth-reduction techniques such as per-peer deduplication are more
efficient because of the smaller message size
* gossip verification happens independently for blocks and blobs,
allowing better sharing / use of CPU and I/O resources in clients
With growing block sizes and additional blob data to stream, the network
streaming time becomes a dominant factor in propagation times - on a
100mbit line, streaming 1mb to 8 peers takes ~1s - this process is
repeated for each hop in both incoming and outgoing directions.
This design in particular sends each blob on a separate subnet, thus
maximising the potential for parallelisation and providing a natural
path for growing the number of blobs per block should the network be
judged to be able to handle it.
Changes compared to the current design include:
* `BlobsSidecar` is split into individual `BlobSidecar` containers -
each container is signed individually by the proposer
* the signature is used during gossip validation but later dropped.
* KZG commitment verification is moved out of the gossip pipeline and
instead done before fork choice addition, when both block and sidecars
have arrived
* clients may verify individual blob commitments earlier
* more generally and similar to block verification, gossip propagation
is performed solely based on trivial consistency checks and proposer
signature verification
* by-root blob requests are done per-blob, so as to retain the ability
to fill in blobs one-by-one assuming clients generally receive blobs
from gossip
* by-range blob requests are done per-block, so as to simplify
historical sync
* range and root requests are limited to `128` entries for both blocks
and blobs - practically, the current higher limit of `1024` for blocks
does not get used and keeping the limits consistent simplifies
implementation - with the merge, block sizes have grown significantly
and clients generally fetch smaller chunks.
2023-02-07 11:05:51 +01:00
0bf006cc06
Merge remote-tracking branch 'upstream/dev' into 3207
2023-02-02 16:34:34 +10:00
659c7f513f
Change blob verification fiat-shamir to single blob
2023-01-29 13:05:02 +00:00
d89e579089
Updater toc
2023-01-27 14:33:43 +00:00
03f4b8fa4d
Add KZG multi verify function
2023-01-27 14:23:38 +00:00
127edf3af1
Merge pull request #3221 from benjaminion/fc-on-attester-slashing
...
Add on_attester_slashing() to handlers list
2023-01-27 21:49:34 +08:00
ce987c4a64
Delete `is_merge_transition_block` check from Capella
2023-01-27 09:12:39 +01:00
6e397b195b
EIP4844: Public methods take bytes as input (explicit validation) ( #3224 )
2023-01-25 16:15:19 +01:00
aede132f49
Merge pull request #3220 from benjaminion/fc-genesis-block
...
Add genesis_block to get_forkchoice_store() params
2023-01-24 22:22:25 +08:00
5eca56901e
Update polynomial-commitments.md ( #3223 )
2023-01-24 15:14:25 +01:00
8a0de1c595
Add on_attester_slashing() to handlers list
2023-01-24 13:25:40 +00:00
d54c87a5c0
Add genesis_block to get_forkchoice_store() params
2023-01-24 13:08:41 +00:00
2626126935
EIP4844: compute_kzg_proof() now takes bytes as input ( #3219 )
2023-01-24 14:02:22 +01:00
dd5c5af117
Merge pull request #3217 from pawanjay176/remove-point-validation
...
Remove kzg point and blob validations in gossip
2023-01-24 10:38:03 +01:00
317facbd64
Replaced EIP4844 references with Deneb
...
Fixes #3207
2023-01-23 15:08:34 +01:00
a0791712cd
Remove kzg point validations in gossip
2023-01-20 21:32:03 +05:30
f1435d456d
Update p2p-interface.md
2023-01-19 21:26:21 +01:00
04d8f28cf6
Merge pull request #3214 from asn-d6/barycentric_no_assert
...
EIP4844: Handle barycentric evaluation at roots of unity
2023-01-17 23:00:17 +08:00
521de12d9b
Merge pull request #3151 from etan-status/lc-eph
...
Add `ExecutionPayloadHeader` to LC data
2023-01-17 21:33:35 +08:00
296ba921c9
EIP4844: Handle barycentric evaluations at roots of unity
2023-01-16 17:51:50 +02:00
514d4431ca
Use `bellatrix` in `upgrade_x_to_capella` helpers
2023-01-13 20:11:05 +01:00
427a6e809f
Merge pull request #3206 from ethereum/btec-fork-version
...
MUST use `GENESIS_FORK_VERSION` to sign `BLSToExecutionChange` message
2023-01-13 23:29:21 +08:00
a580f82c7d
Use `beacon` wrapper in `upgrade_lc_header_to_capella`
2023-01-13 14:59:40 +01:00
ca32fe8347
Add docstrings to explain empty header
2023-01-13 11:19:53 +01:00
6a19cf568a
PR feedback from @terencechain: re-use `current_epoch` definition
2023-01-13 16:49:24 +08:00
82d6267951
Merge branch 'dev' into lc-eph
2023-01-12 18:44:54 +01:00
2b2ffb5229
Merge pull request #3190 from etan-status/lc-headerwrapper
...
Add `LightClientHeader` wrapper
2023-01-13 00:40:44 +08:00
4af46e7e78
Ignore `bls_to_execution_change` messages before `CAPELLA_FORK_EPOCH`
2023-01-12 23:47:36 +08:00
730a7afe49
Always use `GENESIS_FORK_VERSION` to sign `BLSToExecutionChange` message
2023-01-12 23:28:41 +08:00
fc6bdade50
Merge pull request #3174 from terencechain/clarify-empty-blob-sidecar
...
EIP4844: Clarify ratelimit behavior for sidecar with zero blobs
2023-01-11 19:45:03 -07:00
8d841c864b
minor copy edit
2023-01-11 19:44:39 -07:00
71d66afc0a
Merge pull request #3201 from ethereum/fix-fork-historical-summaries
...
Add `historical_summaries` setup to `fork.md`
2023-01-11 13:24:55 -07:00
96a553615a
Add rate limit suggestion
2023-01-10 10:28:19 -08:00
53a95f0007
Merge branch 'dev' into lc-eph
2023-01-10 13:43:15 +01:00
f057f530fe
Add `historical_summaries` setup to fork.md
2023-01-10 16:45:58 +08:00
d23d0147c8
Update p2p-interface.md
...
fixed typo
2023-01-07 14:28:12 -06:00
b951ce077a
Move helper
2023-01-06 20:01:02 +01:00
5230c1ba87
Add `is_valid_light_client_header` and comments
2023-01-06 19:58:23 +01:00
79a717c8f6
Merge branch 'dev' into lc-headerwrapper
2023-01-06 19:45:18 +01:00
ed1c6be132
Merge pull request #3185 from ethereum/fc-is_data_available
...
Move `is_data_available` check to fork-choice `on_block`
2023-01-06 10:55:29 -05:00
4ba2266fd5
Update specs/eip4844/fork-choice.md
2023-01-06 08:45:20 -07:00
c9f8e4fef8
Move `BlobsSidecar`
2023-01-06 23:39:04 +08:00
e154b3414c
Move `validate_blobs_sidecar` to fork-choice and add spec into execution spec scope
2023-01-06 23:19:44 +08:00
d5c24dc731
Merge pull request #3186 from ethereum/capella-note
...
Add notes for new state historical accumulators
2023-01-06 10:13:32 -05:00
75937e5b55
Merge pull request #3181 from terencechain/4844-enable-withdrawal
...
EIP4844: Enable withdrawal
2023-01-06 10:12:57 -05:00
d679b2e80d
Update specs/capella/beacon-chain.md
2023-01-06 08:06:39 -07:00
e937e2abbc
Apply suggestions from code review
...
Co-authored-by: Danny Ryan <dannyjryan@gmail.com>
2023-01-06 13:08:32 +08:00
375436094c
Merge branch 'dev' into lc-eph
2023-01-05 13:53:29 +01:00
5c64a2047a
Add `LightClientHeader` wrapper
...
In Altair, light client sync protocol exchanges `BeaconBlockHeader`
structures for tracking current progress. Wrapping `BeaconBlockHeader`
inside a `LightClientHeader` allows future extensions of this header,
e.g., to also track `ExecutionPayloadHeader`.
Note: This changes the JSON REST format by adding a `beacon` nesting.
For SSZ, the serialization format stays same (but overall root changes).
2023-01-05 13:36:04 +01:00
19cf15b838
Merge pull request #3188 from henridf/patch-2
...
Fix typo in eip4844/BeaconBlocksByRoot docs
2023-01-05 12:52:14 +08:00
0777a52f55
Merge pull request #3149 from etan-status/lc-toheader
...
Add `block_to_light_client_header` helper
2023-01-05 12:12:36 +08:00
7d2482a025
Fix typo in eip4844/BeaconBlocksByRoot docs
2023-01-04 16:29:15 +01:00
118daae6d9
Add notes for new state historical accumulators
2023-01-04 17:55:51 +08:00
bb3601dcfa
Merge branch 'dev' into pr3181
2023-01-04 01:07:30 +08:00
0f07cdcf72
Merge pull request #3165 from arnetheduck/block-root-accumulator-4
...
Historical batches
2023-01-04 00:23:21 +08:00
6b94aab3af
Move `is_data_available` check to fork-choice `on_block`
2023-01-04 00:15:15 +08:00
b0471511e5
Merge branch 'dev' into lc-eph
2023-01-03 15:36:37 +01:00
dcacb7164f
Rename `block_batch_root` to `block_summary_root` and `state_batch_root` to `state_summary_root`
2023-01-03 21:50:06 +08:00
1cfabcbe54
Rename `HistoricalBatchSummary` -> `HistoricalSummary` and `historical_batches` -> `historical_summaries`
2023-01-02 23:07:00 +08:00
280f6e0568
Merge pull request #3167 from ethereum/akzg-tp
...
Add `compute_kzg_proof` as a public method
2023-01-01 14:14:53 -06:00
4c573ff6b9
Add compute_kzg_proof to public methods
2023-01-01 14:04:56 -06:00
2afb02ba30
Fix Toc
2022-12-28 18:49:02 -08:00
45b1eb7979
EIP4844: Enable withdrawal
2022-12-28 18:42:24 -08:00
6eb118d34a
Fix finality check in deposit processing
2022-12-24 09:40:15 +06:00
48f120c90e
Set a higher limit to deposit receipts list
2022-12-24 09:40:15 +06:00
9d2a8f7d63
Give deposits EIP a name
2022-12-24 09:40:15 +06:00
8cc293c869
Fix wording
2022-12-24 09:40:15 +06:00
5ea983ac33
Fix toc, add more comments
2022-12-24 09:40:15 +06:00
b3c771c46d
Preserve deposits per epoch boundary
2022-12-24 09:40:15 +06:00
c53ab45e19
Add in-protocol deposit processing
2022-12-24 09:40:15 +06:00
7204b5c3aa
Merge branch 'clarify-empty-blob-sidecar' of github.com:terencechain/eth2.0-specs into clarify-empty-blob-sidecar
2022-12-23 22:24:40 +08:00
491bb1d207
Clean ups
2022-12-23 22:21:51 +08:00
518c49bbba
Update specs/eip4844/p2p-interface.md
...
Co-authored-by: g11tech <develop@g11tech.io>
2022-12-23 20:47:14 +08:00
42121adeb0
Clarify what empty means
2022-12-23 14:30:30 +08:00
5a75704e37
Clarify behavior for empty blobs sidecar
2022-12-23 14:02:33 +08:00
4f77493b0f
Merge pull request #3169 from ethereum/false-after-prune
...
Handle unavailable data outside of prune window
2022-12-20 17:15:17 -07:00
e6f5ab9a16
Merge pull request #3154 from dapplion/blobs-byroot-errorcode
...
Clarify BeaconBlockAndBlobsSidecarByRoot no blob available
2022-12-20 17:14:39 -07:00
b6ac1fa099
Update specs/capella/beacon-chain.md
...
Co-authored-by: Danny Ryan <dannyjryan@gmail.com>
2022-12-20 07:46:18 +01:00
4be8f7d669
clarify how to handle unavailable data outside of the prune window
2022-12-19 10:15:18 -07:00
84592f101b
Add tests for `process_historical_batches_update`
2022-12-16 01:01:00 +08:00
c185f91e00
Merge branch 'dev' into pr2649-tests
2022-12-15 22:56:04 +08:00
fac4b49a9b
Change range lower bound to EIP4844_FORK_EPOCH
2022-12-15 11:27:33 +07:00
eaa1268dc4
Place dominant factor first
2022-12-15 11:12:37 +07:00
29d36d8a78
Merge two sentences + drop range
2022-12-15 11:11:15 +07:00
05c2eb8ace
Merge pull request #3159 from kevaundray/patch-3
...
EIP4844: Clarify that z should not be in the domain
2022-12-14 17:47:48 +02:00
f7187a6657
Merge pull request #3156 from ethereum/kzg-trusted-setup
...
Update G2 trusted setup length to 65
2022-12-14 08:15:53 -07:00
e467ab9556
clarify that z should not be in the domain
2022-12-14 11:01:40 +00:00
224b0f20cf
Add missing qoute
2022-12-13 11:53:18 -06:00
26605ac724
Fix some nits in eip4844 specs
2022-12-13 10:07:37 -06:00
02abdc311f
Merge branch 'dev' into lc-eph
2022-12-13 15:33:28 +01:00
3bfac0e16e
Merge branch 'lc-toheader' into lc-eph
2022-12-13 12:31:12 +01:00
ef2a8b319d
Avoid line continuation syntax
2022-12-13 12:30:14 +01:00
2ac06c10d3
Update G2 trusted setup length to 65
2022-12-13 18:14:50 +08:00
da3f5af919
Test case naming clean up ( #3143 )
...
* 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`
2022-12-13 15:13:44 +08:00
06d6d3888f
Merge pull request #3095 from ralexstokes/bound-withdrawals-sweep
...
bound the maximum number of validators considered for withdrawals per sweep
2022-12-12 19:53:29 -07:00
d4b8ab1173
Clarify BeaconBlockAndBlobsSidecarByRoot no blob available
2022-12-13 09:29:28 +07:00
1f943f0d64
Update specs/capella/beacon-chain.md
2022-12-12 14:22:58 -07:00
a8449fa4aa
Fix typo in BlobsSidecarsByRange v1
2022-12-12 15:36:51 +02:00
700bef7a45
Fix `is_valid_light_client_header`
2022-12-12 13:39:14 +01:00
8ad6810a44
EIP4844 support (`excess_data_gas`), fork tests nyi
2022-12-12 13:11:49 +01:00
e67ca3d309
Compute epoch only once for better readability
2022-12-12 13:09:18 +01:00
5028a806ad
Implicit init during fork transition
2022-12-12 12:45:43 +01:00
11d2a59948
Flip `is_valid_light_client_header` logic for extensibility
2022-12-12 12:33:58 +01:00
2e97af2627
Add `ExecutionPayloadHeader` to LC data
...
While the light client sync protocol currently provides access to the
latest `BeaconBlockHeader`, obtaining the matching execution data needs
workarounds such as downloading the full block.
Having ready access to the EL state root simplifies use cases that need
a way to cross-check `eth_getProof` responses against LC data.
Access to `block_hash` unlocks scenarios where a CL light client drives
an EL without `engine_newPayload`. As of Altair, only the CL beacon
block root is available, but the EL block hash is needed for engine API.
Other fields in the `ExecutionPayloadHeader` such as `logs_bloom` may
allow light client applications to monitor blocks for local interest,
e.g. for transfers affecting a certain wallet. This enables to download
only the few relevant blocks instead of every single one.
A new `LightClientStore` is proposed into the Capella spec that may be
used to sync LC data that includes execution data. Existing pre-Capella
LC data will remain as is, but can be locally upgraded before feeding it
into the new `LightClientStore` so that light clients do not need to run
a potentially expensive fork transition at a specific time. This enables
the `LightClientStore` to be upgraded at a use case dependent timing at
any time before Capella hits. Smart contract and embedded deployments
benefit from reduced code size and do not need synchronization with the
beacon chain clock to perform the Capella fork.
2022-12-12 00:48:40 +01:00
9ab22b3ef3
Merge branch 'lc-accessors' into lc-eph
2022-12-12 00:41:31 +01:00
37e504e784
bound the maximum number of validators considered for withdrawals per sweep
2022-12-09 10:51:25 -07:00
14fd937046
Fix
2022-12-06 23:21:06 +01:00
0fb0b26742
Add accessors for LC header
...
Introduce `get_lc_beacon_slot` and `get_lc_beacon_root` accessors
similar to `get_current_slot(state)` to account for future extensions
to the light client header structure that may override how those fields
are accessed. Idea is to extend with execution accessors in the future.
2022-12-06 22:05:19 +01:00
8bf801ecc6
Add `block_to_light_client_header` helper
...
Introduce `block_to_light_client_header` helper function to enable
future forks to override it with additional info (e.g., execution),
without having to change the general light client logic.
Likewise, update existing light client data creation flow to use
`block_to_light_client_header` and default-initialize empty fields.
Furthermore, generalize `create_update` helper to streamline test code
using `block_to_light_client_header`.
Note: In Altair spec, LC header is the same as `BeaconBlockHeader`.
however; future forks will extend it with more information.
2022-12-06 17:44:41 +01:00
9839ed4934
Merge pull request #3145 from terencechain/update-max-blobs
...
EIP4844: Update `MAX_BLOBS_PER_BLOCK` to 4
2022-12-06 09:37:05 -07:00
5498519945
Merge pull request #3125 from terencechain/rm-is-data-available
...
EIP4844: remove optimistic sync mentions
2022-12-06 09:29:34 -07:00
f4dafcfebc
Merge branch 'dev' into lc-blockfuncs
2022-12-05 18:38:40 +01:00
23d3aeebba
EIP4844 precompile: Be strict when parsing field elements from network ( #3138 )
...
* EIP4844: bytes_to_bls_field() must not accept values >= BLS_MODULUS
bytes_to_bls_field() will be used in the precompile and hence it should error out when provided with malicious inputs.
* EIP4844: Add hash_to_bls_field() for use in compute_challenges()
The previous commit made bytes_to_bls_field() be strict about its inputs. However in compute_challenges() we are
dealing with Fiat-Shamir and hash outputs that could be innocuously higher than the modulus. For this reason we add the
hash_to_bls_field() helper for use in compute_challenges().
* EIP4844: Further use of bytes_to_bls_field() // Fix executable spec
Co-authored-by: Hsiao-Wei Wang <hsiaowei.eth@gmail.com>
2022-12-01 14:59:00 +02:00
602f846dc7
Merge pull request #3134 from ethereum/eip4844-lint
...
Add EIP4844 pylint and Mypy checks
2022-12-01 20:01:48 +08:00
aae00a0bf6
Clarify that full withdrawals only apply to exited validators
2022-11-30 16:21:06 -08:00
5b0bfe7eb6
EIP4844: Update `MAX_BLOBS_PER_BLOCK` to 4
2022-11-30 14:34:09 -08:00
b62c9e8779
Merge pull request #3014 from tersec/deprecate-phase0-beacon-blocks-req-resp
...
deprecate phase 0-only versions of beacon_blocks_by_{range,root}
2022-11-29 22:09:13 +08:00
339b788839
Revert previous changes. Remove optimistic wording
2022-11-28 19:52:11 -08:00
b3a176689d
WIP. Fixing mypy errors
2022-11-28 21:01:41 +08:00
4f32fd95d6
Enable EIP4844 lint and fix Pylint
2022-11-28 20:01:50 +08:00
a1e46d1ae4
EIP4844: Fix return value of compute_challenges ( #3129 )
...
Co-authored-by: George Kadianakis <desnacked@riseup.net>
2022-11-24 15:26:02 +02:00
3ec25a8755
Fix docstring for compute_powers and compute_challenges methods ( #3130 )
2022-11-24 15:11:46 +02:00
f0ff15c14d
EIP4844: Refactor Fiat-Shamir logic to handle empty sidecars ( #3093 )
...
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>
2022-11-23 16:52:47 +02:00
3624742ff5
Remove is_data_available
2022-11-22 09:15:17 -08:00
4e3b700b86
Fix initialization of VERSIONED_HASH_VERSION_KZG
2022-11-22 11:51:10 +04:00
0fb2447a95
Merge branch 'dev' into lc-blockfuncs
2022-11-19 12:20:18 +01:00
19c8f21b63
Fix type annotation in verify_aggregate_kzg_proof
2022-11-19 09:17:11 +00:00
a45627164d
Merge pull request #3097 from asn-d6/verify_kzg_proof_bytes_api
...
EIP4844: Refactor `verify_kzg_proof()` to receive bytes (used in precompile)
2022-11-18 20:01:18 +02:00
46c0447662
Merge pull request #3110 from ethereum/eip4844-test-vectors
...
Add EIP4844 to testgen and add more Capella+EIP4844 random fork tests
2022-11-18 17:18:02 +08:00
30e975806f
Merge pull request #3108 from realbigsean/validate-blobs-sidecar-in-gossip
...
Make `validate_blobs_sidecar` a gossip condition
2022-11-17 16:31:14 -07:00
dd8b67075e
Add note comment
2022-11-18 04:51:19 +08:00
438a7e55ea
Conver `roots_of_unity_brp` to integers
2022-11-18 03:20:25 +08:00
78f0e0301e
Merge pull request #3052 from Inphi/inphi/eip4844-rebase
...
Rebase EIP-4844 on Capella
2022-11-18 03:16:58 +08:00
de789e2d5b
Merge pull request #3090 from ethereum/4844-topic-dep
...
clarify deprecation of `beacon_block` gossip topic in 4844
2022-11-17 12:16:24 -07:00
ee0e2a03d2
Merge branch 'dev' into pr3052
2022-11-18 02:44:46 +08:00
30dce7f232
Apply suggestions from code review
...
Co-authored-by: Alex Stokes <r.alex.stokes@gmail.com>
2022-11-18 02:25:56 +08:00
f1d4c9047a
Merge branch 'dev' into pr3052
2022-11-18 02:12:07 +08:00
620943b275
Update specs/eip4844/fork.md
...
Co-authored-by: Alex Stokes <r.alex.stokes@gmail.com>
2022-11-16 19:19:39 +01:00
e8e860e1f3
make a gossip condition
2022-11-16 10:46:04 -05:00
b737e5370f
fix state upgrade
2022-11-16 12:12:18 +01:00
a5f8a15819
Clarify how to transition gossip
2022-11-15 10:29:03 -07:00
11a037fd92
Merge pull request #3089 from terencechain/beacon-block-and-blobs-sidecar-by-root
...
EIP4844: Add block and sidecar retrival by root
2022-11-16 00:41:58 +08:00
bbc3ec02ff
Fix typos
2022-11-12 23:36:34 -08:00
fcac0b5c69
Clarify which functions are public and provided by the KZG library
2022-11-12 01:47:39 +02:00
ff34006807
Refactor `verify_kzg_proof()` to receive bytes (used in precompile)
...
This way, client devs don't need to convert to field elements themselves, and the KZG library takes care fo it.
2022-11-12 01:47:39 +02:00
3df1371897
Merge remote-tracking branch 'origin/dev' into inphi/eip4844-rebase
2022-11-11 17:32:06 -05:00
cd1e1133a5
excess_data_gas uint256
2022-11-11 17:25:06 -05:00
6327ffa687
rename excess_blobs
2022-11-11 17:24:05 -05:00
38669d4b7a
Update specs/eip4844/p2p-interface.md
...
Co-authored-by: Danny Ryan <dannyjryan@gmail.com>
2022-11-11 11:26:48 -07:00
078bba2e3c
clarify deprecation of `beacon_block` gossip topic in 4844
2022-11-11 11:26:45 -07:00
faf308349e
Merge pull request #3086 from michaelsproul/light-client-p2p-fix
...
Fix slot checks in light client p2p spec
2022-11-11 11:03:33 -07:00
e2a2d68cae
Added a comment to support pre-fork-epoch
2022-11-11 09:50:06 -08:00
897b97990f
Further clarify light client slot checks
2022-11-11 22:09:33 +11:00
104cba06a6
replace get_blobs_and_kzg_commitments
2022-11-11 00:21:56 -05:00
bed1df00a2
Remove withdrawal_queue from BeaconState upgrade
2022-11-10 13:57:30 -05:00
d4a9b97cd0
Merge pull request #3084 from ethereum/validator-note-for-bls-change
...
Add note to Capella validator guide about BLS to execution change
2022-11-10 11:32:04 -07:00
a59dd37933
Merge remote-tracking branch 'origin/dev' into inphi/eip4844-rebase
2022-11-10 13:17:57 -05:00
4b573f82f6
Merge pull request #3091 from terencechain/patch-138
...
EIP4844: Explicitly mention old gossip validations for block
2022-11-10 11:16:18 -07:00
0e7d3555ea
minor copy edits
2022-11-10 11:12:33 -07:00
37996fc36c
Merge pull request #3068 from potuz/withdrawals_without_queues
...
Withdrawals without queues
2022-11-10 10:49:25 -07:00
be85e9356a
Update specs/capella/validator.md
...
Co-authored-by: Hsiao-Wei Wang <hsiaowei.eth@gmail.com>
2022-11-10 09:12:35 -05:00
0f93e9aad8
Update specs/capella/validator.md
...
Co-authored-by: Hsiao-Wei Wang <hsiaowei.eth@gmail.com>
2022-11-10 09:12:10 -05:00
be098b60c0
Update specs/eip4844/p2p-interface.md
2022-11-10 09:10:01 -05:00
7f266bcb0f
Use next_validator_withdrawal_index
2022-11-10 08:33:11 -03:00
579f8fd1d2
EIP4844: Explicitly mention old gossip validations for block
2022-11-09 15:06:58 -08:00
710b124cdc
fix last commit
2022-11-08 21:51:54 -03:00
8f42e485c7
Add extra %
2022-11-08 19:53:58 -03:00
dae1011194
Fix typos
...
Co-authored-by: Alex Stokes <r.alex.stokes@gmail.com>
2022-11-09 09:41:21 +11:00
c447662ba0
Fix `BeaconBlocksByRootV2` for pre-4844 support
2022-11-08 12:55:43 -08:00
cb9159fe58
EIP4844: Add block and sidecar retrival by root
2022-11-08 10:41:48 -08:00
e3e73a8f54
Add Capella fork version
2022-11-08 09:06:38 -05:00
91de8a09f3
Fix slot checks in light client p2p spec
2022-11-08 17:06:07 +11:00
8bb1751f46
Add note to Capella validator guide about BLS to execution change
2022-11-07 15:22:26 -07:00
99e2704c2c
make tests pass
2022-11-07 17:56:01 -03:00
ac670e2c47
add phase0 sentence regarding process slots
2022-11-07 17:34:19 -03:00
2f89f5096d
Danny's review 1st pass
2022-11-07 17:29:56 -03:00
f010de0871
Remove domain blobs sidecar
2022-11-07 07:16:42 -08:00
95ee291f58
Merge branch 'dev' into pr3052
2022-11-07 10:10:16 -05:00
544ae28721
Merge pull request #3016 from ethereum/gen_kzg_trusted_setups
...
Rework the testing KZG trusted setups presets
2022-11-07 22:29:54 +08:00
b7dfd5a37e
Merge pull request #3027 from ethereum/eip-4844-fee-ssz-fix
...
eip4844: update tx_peek_blob_versioned_hashes to match tx type from fee market update
2022-11-07 22:22:12 +08:00
dac756efee
Minor clean up
2022-11-07 09:09:09 -05:00
08ff71688f
Merge branch 'dev' into lc-blockfuncs
2022-11-07 15:03:20 +01:00
72544d5307
Merge pull request #3081 from ethereum/add-capella-bls-to-exec-change-block-packing
...
add note about packing BLS to execution changes into a block
2022-11-07 06:22:04 +08:00
6f0a7ef046
Merge pull request #3080 from ethereum/ralexstokes-patch-1
...
Fix typo in `bellatrix` p2p docs
2022-11-07 05:18:03 +08:00
b5ecf44c91
add note about packing BLS to execution changes into a block
2022-11-04 16:41:07 -06:00
e1e37deb56
Merge pull request #3075 from ethDreamer/gossip_bls_to_execution_change
...
Capella: Add Gossip Topic bls_to_execution_change
2022-11-04 16:22:43 -06:00
c8d1614e7e
Apply suggestions from code review
2022-11-04 16:19:26 -06:00
355ca296b8
Update specs/capella/p2p-interface.md
...
Co-authored-by: Alex Stokes <r.alex.stokes@gmail.com>
2022-11-04 17:13:23 -05:00
6c1fa0a485
Update specs/capella/p2p-interface.md
...
Co-authored-by: Alex Stokes <r.alex.stokes@gmail.com>
2022-11-04 17:13:15 -05:00
1f68c57a7b
Update specs/capella/p2p-interface.md
...
Co-authored-by: Alex Stokes <r.alex.stokes@gmail.com>
2022-11-04 17:13:07 -05:00
cd5567b735
Update p2p-interface.md
2022-11-04 15:54:30 -06:00
8488fb79d9
Alex Stokes' review
2022-11-04 18:47:56 -03:00
e15b02d16f
lint
2022-11-04 09:13:21 -03:00
22f803a038
name change and increment operator
2022-11-04 07:20:26 -03:00
6e913ecbd2
rename to latest_withdrawal_validator_index
2022-11-03 20:23:35 -03:00
0b1f32ec25
Fix capella/block_processing tests
2022-11-03 16:08:05 -05:00
a09d617737
right ret
2022-11-03 15:07:03 -03:00
329bafa6e2
dapplion's suggestions
2022-11-03 14:58:55 -03:00
12404d0250
fix for loop
2022-11-03 14:56:38 -03:00
ad36548480
Fix auto-rebase errors
2022-11-03 11:46:07 -05:00
39e6ec5d14
lint
2022-11-03 11:43:56 -05:00
a14479a70f
g11tech review
2022-11-03 11:43:56 -05:00
ff1dd90012
fix toc
2022-11-03 11:43:24 -05:00
5b92eae08f
type annotation
2022-11-03 11:43:24 -05:00
0f74ab5686
type annotation
2022-11-03 11:43:24 -05:00
c156ea6adc
linting and typo
2022-11-03 11:43:24 -05:00
49a2519da2
lint
2022-11-03 11:43:24 -05:00
7dbd50e958
Reviewers' comments
...
- Implemented many of Alex's comments including reinsertion of the
withdrawal index in the BeaconState
- Implemented Sean's suggestion of separating the logic for block
production so that one matches the list in the payload with what
`get_expected_withdrawals` returns
- Changed `get_expected_wihdrawals` to match the current behavior and
moved it to `beacon-chain.md`
2022-11-03 11:42:31 -05:00
f506087af5
rebase on top of develop
2022-11-03 11:42:30 -05:00
d958ed70b4
Implement withdrawals without queues
2022-11-03 11:40:54 -05:00
b2d72a18f0
Fix type error in the inputs to hash_to_bls_field()
2022-11-03 18:13:36 +02:00
5a0d84ed01
int casting to avoid underflow
2022-11-03 10:21:09 -05:00
86e15764ad
EIP4844: Update cryptography API ( #3038 )
...
This commit changes the public API of the KZG library to the following high-level API:
```
- verify_kzg_proof()
- compute_aggregate_kzg_proof()
- verify_aggregate_kzg_proof()
- blob_to_kzg_commitment()
```
compared to the previous much more low-level API:
```
- compute_powers()
- matrix_lincomb()
- lincomb()
- bytes_to_bls_field()
- evaluate_polynomial_in_evaluation_form()
- verify_kzg_proof()
- compute_kzg_proof()
```
This means that all the cryptographic logic (including Fiat-Shamir) is now isolated and hidden in the KZG library and the `validator.md` file ends up being significantly simplified, only calling high-level KZG functions.
Some additional things that this commit does:
- Moves all EIP4844 cryptography into polynomial-commitments.md
- Improves the Fiat-Shamir stack by removing the need for SSZ and by introducing simple domain separators
Co-authored-by: Kevaundray Wedderburn <kevtheappdev@gmail.com>
Co-authored-by: Hsiao-Wei Wang <hsiaowei.eth@gmail.com>
Co-authored-by: Dankrad Feist <mail@dankradfeist.de>
2022-11-03 17:01:32 +02:00
87dcb83827
Merge pull request #3063 from etan-status/lc-gossipdigests
...
Document how to derive fork context for LC gossip
2022-11-02 13:12:23 +08:00
826c20ffc2
Capella: Add Gossip Topic bls_to_execution_change
2022-11-01 12:54:33 -05:00
498a5b0722
Merge pull request #3062 from terencechain/rm-signed-blobs
...
EIP4844: Remove signed blobs
2022-11-01 22:32:45 +08:00
ff0e525ecc
Pass blocks to LC data creation functions
...
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.
2022-10-28 22:14:25 +02:00
be3c774069
Merge pull request #3065 from etan-status/lc-branchrootdocs
...
Clarify where LC proofs are rooted
2022-10-28 08:51:26 -06:00
5b6eb8007a
Clarify where LC proofs are rooted
...
Add more detailed LC object documentation to explain that the various
merkle proofs are relative to the beacon block's state root.
Likewise, clarify that sync committees relate to the finalized header
(not to the optimistic header, which can be a period ahead).
2022-10-27 21:44:38 +02:00
af54c97a6c
Consistently use `compute_sync_committee_period_at_slot`
...
A few LC functions were not yet updated to use a more concise function
for computing sync committee period for a slot. Updating to that func.
2022-10-27 21:36:55 +02:00
c399e11aa7
Document how to derive fork context for LC gossip
...
For LC gossip, the documentation did not specify what slot number to use
for deriving the gossip objects. This missing documentation is now added
to document using `attested_header.slot`.
2022-10-27 21:23:09 +02:00
71775c6e87
EIP4844: Remove signed blobs
2022-10-27 08:10:32 -07:00
4d1b487b21
move field last
...
avoids changing "header" fields in state
2022-10-27 10:38:09 +02:00
1c8d57eeb7
Historical batches
...
This PR, a continuation of
replaces `historical_roots` with
`historical_block_roots`.
By keeping an accumulator of historical block roots in the state, it
becomes possible to validate the entire block history that led up to
that particular state without executing the transitions, and without
checking them one by one in backwards order using a parent chain.
This is interesting for archival purposes as well as when implementing
sync protocols that can verify chunks of blocks quickly, meaning they
can be downloaded in any order.
It's also useful as it provides a canonical hash by which such chunks of
blocks can be named, with a direct reference in the state.
In this PR, `historical_roots` is frozen at its current value and
`historical_batches` are computed from the merge epoch onwards.
After this PR, `block_batch_root` in the state can be used to verify an
era of blocks against the state with a simple root check.
The `historical_roots` values on the other hand can be used to verify
that a constant distributed with clients is valid for a particular
state, and therefore extends the block validation all the way back to
genesis without backfilling `block_batch_root` and without introducing
any new security assumptions in the client.
As far as naming goes, it's convenient to talk about an "era" being 8192
slots ~= 1.14 days. The 8192 number comes from the
SLOTS_PER_HISTORICAL_ROOT constant.
With multiple easily verifable blocks in a file, it becomes trivial to
offload block history to out-of-protocol transfer methods (bittorrent /
ftp / whatever) - including execution payloads, paving the way for a
future in which clients purge block history in p2p.
This PR can be applied along with the merge which simplifies payload
distribution from the get-go. Both execution and consensus clients
benefit because from the merge onwards, they both need to be able to
supply ranges of blocks in the sync protocol from what effectively is
"cold storage".
Another possibility is to include it in a future cleanup PR - this
complicates the "cold storage" mode above by not covering exection
payloads from start.
2022-10-27 10:33:38 +02:00
af8c5bf532
Merge pull request #3046 from terencechain/couple-block-sidecar
...
EIP4844: couple beacon block and blob sidecar for p2p
2022-10-26 23:51:23 +08:00
b7c72c3b42
Merge pull request #3047 from terencechain/reduce-min-epochs-blobs-reqs
...
EIP4844: modify `MIN_EPOCHS_FOR_BLOBS_SIDECARS_REQUESTS` to 18 days
2022-10-26 23:26:45 +08:00
ca538f52a0
Update specs/eip4844/beacon-chain.md
...
Co-authored-by: Alex Stokes <r.alex.stokes@gmail.com>
2022-10-25 12:06:07 -04:00
459310f744
Fix test_process_execution_payload
2022-10-24 16:25:50 -04:00
0488c0c26e
remove unchanged epoch processing section
2022-10-24 12:51:54 -04:00
2ac57c7e7b
Fix py setup
...
And remove the ENABLE_WITHDRAWALS feature-flag. The Testing section in
the spec has been updated to specify how withdrawals is to be disabled
2022-10-24 12:44:11 -04:00
9844ea1f30
Merge pull request #3040 from ethereum/capella-test-vectors
...
Capella test vectors
2022-10-23 06:44:03 +08:00
e2d0a4fa9a
Fix ToC and test
2022-10-22 08:36:46 -07:00
9f4ae4c6ee
Update validator spec with `SignedBeaconBlockAndBlobsSidecar`
2022-10-21 17:33:55 -07:00
f6f2474c87
Update specs/eip4844/beacon-chain.md
...
Co-authored-by: Hsiao-Wei Wang <hsiaowei.eth@gmail.com>
2022-10-20 14:39:18 -04:00
242e1b73bb
Rebase Capella on EIP-4844
...
This also introduces an `ENABLE_WITHDRAWALS` feature-flag to allow
implementers test EIP-4844 without including Capella-specific state
changes.
2022-10-20 10:36:08 -04:00
53b63cedc5
Merge branch 'dev' into pr3050
2022-10-19 10:21:32 -05:00
dfa5ac8008
@protolambda's feedback, `SignedBlobsSidecar` -> `BlobsSidecar`
2022-10-18 11:34:14 -07:00
cab2e3ef9a
Describe sidecar and block are recieved together
...
Co-authored-by: protolambda <proto@protolambda.com>
2022-10-18 11:23:19 -07:00
779d9be66a
Compare sidecar slot with block slot
...
Co-authored-by: protolambda <proto@protolambda.com>
2022-10-18 11:22:43 -07:00
0345b2284a
Add validator_index to Withdrawal.
2022-10-18 15:35:06 +01:00
b128148a17
Modify `MIN_EPOCHS_FOR_BLOBS_SIDECARS_REQUESTS` to 18 days
2022-10-17 16:24:19 -07:00
96b90200b7
Fix ToC
2022-10-17 16:08:59 -07:00
eb5a2c2741
Couple beacon block and blob sidecar for p2p
2022-10-17 15:16:38 -07:00
b90436c988
Fix capella random & fork
2022-10-14 23:42:42 -05:00
24f8ec170b
Remove work-in-progress notes in Bellatrix specs ( #3033 )
...
* Remove the work-in-progress note in Bellatrix spec
Bellatrix is done and released.
* Remove work-in-progress notes in Bellatrix specs
* Remove work-in-progress notes in Bellatrix specs
* Remove work-in-progress notes in Bellatrix specs
2022-10-15 05:05:45 +08:00
092617ec1e
Remove duplicated definition of `Validator`
2022-10-09 17:02:50 -05:00
3552e2f6e8
Merge pull request #3018 from ethereum/bump-dep
...
Bump dep packages version and fix lint issues
2022-10-07 00:21:51 +08:00
63ab5996f4
eip4844: update tx_peek_blob_versioned_hashes to match tx type as defined in EIP PR 5707 (fee market update)
2022-10-06 18:13:24 +02:00
4e26813c81
Merge pull request #3023 from dapplion/patch-1
...
Typo in sync committee duties description
2022-10-05 20:10:12 -06:00
fda2a69ff7
Typo in sync committee duties description
...
I assume it refers to producing a signature for the last slot of phase0 fork
2022-10-02 12:19:55 +02:00
807650e8a1
Fix some type annotations for blobs
...
blobs should not be Sequence[BLSFieldElement], it should be
Sequence[Sequence[BLSFieldElement]]. But we can be more specific and use
Sequence[Blob].
2022-10-01 15:25:47 +01:00
6b9e65fab6
Merge pull request #3019 from terencechain/fix-eip4844
...
EIP4844: various fixes
2022-09-30 10:51:42 -06:00
9dfb1897a4
Merge pull request #3004 from Inphi/inphi/eip4844-fee-mkt
...
EIP-4844: Update ExecutionPayloadHeader
2022-09-30 08:16:53 -06:00
779e6e7b7e
Update specs/eip4844/p2p-interface.md
...
Co-authored-by: Lion - dapplion <35266934+dapplion@users.noreply.github.com>
2022-09-29 13:43:28 -07:00
06c91e4843
Try a different format for epoch
2022-09-29 07:55:50 -07:00
93cb17dc7f
Eip4844: various fixes
2022-09-29 07:36:39 -07:00
44424c4080
Bump dep packages version and fix lint issues
2022-09-28 12:38:21 +08:00
9ac605cc5e
Fix incorrect comment
...
bit-reversal permutation is not idempotent
2022-09-27 12:13:56 +01:00
dapplion
Roberto Saltini
Hsiao-Wei Wang
omahs
terence tsao
George Kadianakis
Danny Ryan
kevaundray
dankrad
Dankrad Feist
Aditya Asgaonkar
Aditya Asgaonkar
Mikhail Kalinin
Ben Edgington
Etan Kissling
Justin Traglia
Stefan Bratanov
Enrico Del Fante
henridf
kasey
Age Manning
Jacek Sieka
Jacek Sieka
Potuz
Paul Harris
protolambda
Pawan Dhananjay
Emilia Hane
GeemoCandama
Justin Traglia
Alex Stokes
Tim Beiko
Roberto Bayardo
Dmitrii Shmatko
Ramana Kumar
realbigsean
Enrico Del Fante
inphi
Michael Sproul
Michael Sproul
ethDreamer
Mark Mackey
Jim McDonald
ethosdev