BlockScience / aztec-gddt

Aztec Granular Design Digital Twin
Apache License 2.0
10 stars 0 forks source link

Check if gas_fee_l1_acceptable = True is fine for s_transactions_new_proposals #282

Closed SeanMcOwen closed 4 months ago

SeanMcOwen commented 4 months ago

Currently is:

gas_fee_l1_acceptable = True #XXX: Temporary economic assumption

We can make a strong assumption but this one feels like a bit too far of an assumption?

jackhack00 commented 4 months ago

This assumption was only set for the limited censorship sim, where we didn't need the more granular censorship option of setting a threshold after which users find it unreasonable to do L1 transactions. With censorship timeseries, we might instead just rework the profitability test alone to reflect gas / blob gas prices being too high

SeanMcOwen commented 4 months ago

Maybe make it a functional parameter and for now have it be a simple always true thing?

SeanMcOwen commented 4 months ago

agent expects profit can be something that is modular, but gas_fee_l1_acceptable can be removed as it is a remnant of censorship

SeanMcOwen commented 4 months ago

@jackhack00 I am also going to get rid of assertions like these unless you tell me not to:

assert ( agent_expects_profit ), "REVEAL_CONTENT: Agent should be expecting profit."

SeanMcOwen commented 4 months ago

Also spawning a tier 5 issue for adding in the non always profitable version: #311