status-im / nimbus-eth2

Nim implementation of the Ethereum Beacon Chain
https://nimbus.guide
Other
523 stars 227 forks source link

add: `recover_matrix` #6553

Closed agnxsh closed 1 week ago

agnxsh commented 1 week ago

This Pr adds the second part to data column reconstruction, where the data columns that would not be unavailable via gossip or req/resp would effectively be reconstructed within the slot period.

ref: https://github.com/ethereum/consensus-specs/blob/v1.5.0-alpha.5/specs/_features/eip7594/das-core.md#recover_matrix

github-actions[bot] commented 1 week ago

Unit Test Results

         9 files  ±0    1 349 suites  ±0   55m 24s :stopwatch: + 5m 40s   5 144 tests +1    4 796 :heavy_check_mark: +1  348 :zzz: ±0  0 :x: ±0  21 204 runs  +3  20 800 :heavy_check_mark: +3  404 :zzz: ±0  0 :x: ±0 

Results for commit 613d4ed1. ± Comparison against base commit b3c8c717.

:recycle: This comment has been updated with latest results.

tersec commented 1 week ago

https://github.com/status-im/nimbus-eth2/pull/6554 fixes the build issues, for a certain definition of fix