Skip to content

Add precompile interfaces for Multi-Constraint Pricer - #19

Merged
eljobe merged 5 commits into
mainfrom
add-precompiles-for-multi-constraint-pricer
Oct 21, 2025
Merged

eljobe merged 5 commits into
mainfrom
add-precompiles-for-multi-constraint-pricer

Conversation

@MishkaRogachev

@MishkaRogachev MishkaRogachev commented Oct 16, 2025 •

Copy link
Copy Markdown
Contributor

Part of NIT-4021

Add Multi-Constraint Pricer methods:

  • Add getGasPricingConstraints to ArbGasInfo to fetch constraints as an array of triples (gas_target_per_second, time_constant_seconds, backlog)
  • Add setGasPricingConstraints to ArbOwner to set constraints as an array of triples (gas target, time constraint, starting_backlog_value)
  • Mark unused methods for Multi-Constraint Price as deprecated
  • Update formatting to satisfy foundry formatter on CI

Comment thread ArbGasInfo.sol
@MishkaRogachev
MishkaRogachev force-pushed the add-precompiles-for-multi-constraint-pricer branch from 7b1e927 to 96ed82d Compare October 17, 2025 14:19
@MishkaRogachev MishkaRogachev assigned eljobe and unassigned gligneul Oct 17, 2025
@MishkaRogachev MishkaRogachev assigned gligneul and unassigned eljobe Oct 17, 2025
@gligneul gligneul assigned eljobe and unassigned gligneul Oct 17, 2025
Comment thread ArbGasInfo.sol
@gligneul gligneul assigned MishkaRogachev and unassigned eljobe Oct 17, 2025
@gzeoneth
gzeoneth self-requested a review October 20, 2025 09:55

@gligneul gligneul left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@gligneul gligneul assigned eljobe and unassigned gligneul Oct 20, 2025
Comment thread ArbGasInfo.sol
@MishkaRogachev MishkaRogachev self-assigned this Oct 21, 2025
@MishkaRogachev
MishkaRogachev marked this pull request as draft October 21, 2025 12:49
@MishkaRogachev
MishkaRogachev marked this pull request as ready for review October 21, 2025 13:40
@gligneul gligneul assigned eljobe and unassigned gligneul Oct 21, 2025
@eljobe
eljobe merged commit 3fc535b into main Oct 21, 2025
1 check passed
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.

3 participants