Skip to content

Conversation

sashass1315
Copy link

  • Deprecate flag throttle.threshold-multiplier; keep backward-compat if --throttle.unsafe-da-bytes-upper-threshold is not set.
  • Add throttle.tx-size-always-limit to cap tx size even at zero intensity.
  • Add throttle.step-threshold-alignment (start|middle|end) to position the step threshold within the [lower, upper] range.
  • Compute throttling intensity from pending DA bytes; still record unsafe DA bytes for visibility; update ratio metric to use pending bytes.
  • Update defaults: lower=2,000,000 bytes (~1–2 channels), upper=lower20 (within 10–50x guidance).
  • Controller: apply always-on tx limit at intensity 0; respect step alignment when selecting threshold.
  • Docs: refresh op-batcher/throttling.md and op-batcher/readme.md with new flags, defaults, and guidance.
  • Minor tests/docs comments adjusted for consistency.

Fix #16947

@sashass1315 sashass1315 requested review from a team as code owners September 12, 2025 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

op-batcher: revisit usability of throttling config
1 participant