SunnysidedJ

Currently under debugging its test case. The test is: https://github.com/sigp/lighthouse/blob/c13e069c9c633535336bca6d6f5f7b30720df629/beacon_node/beacon_chain/tests/store_tests.rs#L2267 However, the test does not run properly when Deneb or Fulu is enabled, e.g. FORK_NAME=deneb cargo test weak_subjectivity_sync --features "fork_from_env".

Status: Ready for review. Tests run: 1. pre-Deneb: as it is in store_tests.rs 2. pre-Fulu, post-Deneb: + incorrect signature in blob header + incorrect KZG commitment in blob 3. post-Fulu:...

Hey @dapplion, I much liked your implementation from https://github.com/sigp/lighthouse/pull/7352, where you've used existing functions I wasn't aware of and also made helper functions that share the same logics between range...

Restructured backfill importing such that 1) it now clearly separates block validation and storing. 2) it follows PeerDAS scoring order

@dapplion could you review the PR?

Major changes: - Separated tests for data column pruning. - Blob pruning tests only run on Deneb while column ones only run on Fulu with PeerDAS enabled. - Found that...

When I enable Fulu and try blob pruning, the blobs aren't pruned correctly. It'd be appreciated if you could have a look on the commit that reproduces the problem: https://github.com/SunnysidedJ/lighthouse/commit/6217cf9d67e3976a0d7d333f3691c316ababaff3

A question left for penalizing peers: https://github.com/sigp/lighthouse/issues/7122#issuecomment-2811455509

Test in progress as commented on the test file

Current status: Issue-specific tests undergoing Implemented as per the latest spec PR but can easily changeable to other options mentioned in the PR