Jimmy Chen

Results 142 comments of Jimmy Chen

I think we should review blob publishing before merging this to avoid adding excessive bandwidth usage for Deneb.

Be great to have https://github.com/sigp/lighthouse/pull/6403 merged first, as there will be some more conflicts that needs to be resolved, and a bit easier in this order. We also need to...

Remaining TODOs - [x] prioritise block publishing (https://github.com/sigp/lighthouse/pull/6268/commits/0a2c6f76c790e2117ed4a225ef312795eb26f240) - [x] send `IDONTWANT` on publish - #6513

I'm observing a few things from testing: The node sometimes end up publishing all blobs, if the pending component is not found in availability cache after the block has been...

@dapplion @michaelsproul I've made a few more changes and have tested this locally. This is ready for review now: - Fix invalid inclusion proof for EL blobs - Only publish...

Some findings from mainnet testing from *a small sample size*: - EL `getBlobs` only gets triggered once roughly every ~5-10 mins. This is likely because relays publish blobs before blocks....

One more thing I was thinking about - we may not want to _always_ fetch blobs on rpc response, as it only makes sense / works if we're close to...

A few other clients have implemented this, and only serve the request if the node already have at least 50% of columns. Fetching 50% / 64 columns from peers isn't...

The peer selection for custody column peer happens here: https://github.com/sigp/lighthouse/blob/90700fe12a28d2795d81a9cbb23cd2b19308d96a/beacon_node/lighthouse_network/src/peer_manager/peerdb.rs#L259-L278