Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Freeze settings v2 #263

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Freeze settings v2 #263

wants to merge 2 commits into from

Conversation

jgur-psyops
Copy link
Contributor

To support arena, we added banks with frozen settings. After some consideration, we have determined that the pool administrator should be able to modify the deposit/borrow limits. Those settings are now configurable even on frozen pools.

Adds a floor to oracle_max_age that is in line with real-world observations of minimum oracle times for txes to actually resolve to avoid a too-low oracle time footgun that renders the pool mostly unusable.

Per discussion, the front end will also enforce the following requirements to be listed on Arena's visible pools:

  • Frozen settings - true
  • Asset/liability weights - within some mrgn-defined limits
  • Deposit/Borrow limits - any, defaults to no cap
  • Operational State - active
  • Oracle - generated by mrgn (a user-generated or user-supplied oracle might be supported if we whitelist it individually, e.g. a sponsored feed)
  • Interest rates - uses mrgn-defined defaults
  • Insurance fees - zero (all losses are socialized)
  • Group fees - within some mrgn-defined range
  • Origination fees - within some mrgn-defined range
  • Risk Tier - Collateral
  • Asset Tag - Default
  • Asset Value Init - no cap
  • oracle age - mrgn-defined value (likely 1-2 minutes)
  • Permissionless debt settlement - true
  • emissions - full configurable, but will not appear on the Arena front end in v1

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.

1 participant