search

ethereum-optimism / specs

OP Stack Specifications
https://specs.optimism.io
Creative Commons Zero v1.0 Universal
102 stars 92 forks source link

isthmus: operator fee #382

Open yuwen01 opened 2 months ago

yuwen01 commented 2 months ago

Overview

We propose adding additional fee scalars to the fee formula, which allow for more flexibility for chains that leverage alt-DA, ZK proving, or custom gas tokens.

This spec goes with this design doc.

tynes commented 1 month ago

This spec is looking great so far! Implementation will want to build on top of https://github.com/ethereum-optimism/optimism/pull/12057 which is targeting audit Nov 4th, so it should be merged in over the next 2 weeks

refcell commented 3 weeks ago

Any updates here?

yuwen01 commented 3 weeks ago

Hi! Sorry for the delay, but I've updated the spec setting the operator fee manager in the Roles struct. See system-config.md in the isthmus folder.

tynes commented 2 weeks ago

This generally looks good to me. Some work started in https://github.com/ethereum-optimism/optimism/pull/12847 to define the config for isthmus that you can build on. cc @vdamle for visibility

tynes commented 2 weeks ago

cc @trianglesphere come leave a review please

yuwen01 commented 4 days ago

This looks good, just add a clarifier for the l1 block attributes function signature so it's clear what that value is at the top of the table in l1-attributes.md.

Thanks! Just fixed.