-
Notifications
You must be signed in to change notification settings - Fork 279
blob sidecar era/erb proposal #5882
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
base: unstable
Are you sure you want to change the base?
Conversation
I was hoping something like this rather than adding blobs to era would be the plan, because it seems cleaner. it sounds like the design is to have
so the 2 archives are aligned - this would make it nice to validate because you can just parse both files. I think that's sensible (ignore above names, just grabbed one i was testing with :) ) Is it worth including the zk proofs from the blocks so that they're at least partially self referencing? I don't think it'd remove the need to cross-validate against the block completely, but it might allow a level of verification of the blob file itself... I guess it increases complexity in a way because then you have to check they're the actual commitments from the block anyway... |
I am cordinating with @arnetheduck @advaita-saha and the specs for e2store will be moved to |
@@ -116,6 +116,14 @@ data: snappyFramed(ssz(BeaconState)) | |||
|
|||
The fork and thus the exact format of the `BeaconState` should be derived from the `slot`. | |||
|
|||
## BlobCompressedSidecars | |||
``` | |||
type: [0x02, 0x00] |
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.
This seems to be the same type
as CompressedBeaconState
? Is this because erb
files are assumed to never contain BeaconState
s?
We've implemented blob import/export in Lighthouse (sigp/lighthouse#6656), but using plain SSZ rather than |
There's some potential value for an Era file equivalent for blob sidecars.
It should intentionally mirror Era file as much as makes sense. Some parts don't, such as emphasizing the states. They're simply not going to be fully verifiable in a standalone way, nor is there much value for their being such.
This version just modified the Era description, as it's easier to work within that regard to start with.
The name is also a stand-in, though,
b
for blobs isn't necessarily bad.Some main real design decisions are
BlobSidecar
s. Arguments for both, but my preference is for SSZs.