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 4fc2724 commit 3322c82
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 3322c82

Please sign in to comment.