Skip to content

Commit

Permalink
Increase MaxFrameLen to 16 MB
Browse files Browse the repository at this point in the history
EigenDA current limit for Holesky (their documentation is currently outdated but the limit seems to be set to 16 MB based on the updated tests from [this PR](Layr-Labs/eigenda-proxy#100)).
  • Loading branch information
jcortejoso authored and karlb committed Oct 14, 2024
1 parent a8b9cfc commit 1089a27
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions op-node/rollup/derive/frame.go
Original file line number Diff line number Diff line change
Expand Up @@ -8,10 +8,10 @@ import (
"io"
)

// Frames cannot be larger than 1 MB.
// Data transactions that carry frames are generally not larger than 128 KB due to L1 network conditions,
// but we leave space to grow larger anyway (gas limit allows for more data).
const MaxFrameLen = 1_000_000
// For AltDA, frames size can be larger. Setting to 16 MB as current blob limit for EigenDA.
const MaxFrameLen = 16_000_000

// Data Format
//
Expand Down Expand Up @@ -85,7 +85,7 @@ func (f *Frame) UnmarshalBinary(r ByteReader) error {
return fmt.Errorf("reading frame_data_length: %w", eofAsUnexpectedMissing(err))
}

// Cap frame length to MaxFrameLen (currently 1MB)
// Cap frame length to MaxFrameLen
if frameLength > MaxFrameLen {
return fmt.Errorf("frame_data_length is too large: %d", frameLength)
}
Expand Down

0 comments on commit 1089a27

Please sign in to comment.