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

[EPIC][CL]: Concentrated Liquidity #3014

Closed
10 tasks done
Tracked by #4700
p0mvn opened this issue Oct 17, 2022 · 2 comments
Closed
10 tasks done
Tracked by #4700

[EPIC][CL]: Concentrated Liquidity #3014

p0mvn opened this issue Oct 17, 2022 · 2 comments
Labels
C:x/gamm Changes, features and bugs related to the gamm module. F: concentrated-liquidity Tracking the development of concentrated liquidity feature to improve filtering on the project board T:epic ⛰

Comments

@p0mvn
Copy link
Member

p0mvn commented Oct 17, 2022

Background

This is an epic to track work on concentrated liquidity (CL). While a concrete direction might still be uncertain, this issue aims to document the latest discussions.

Suggested Design

Introduction

We will begin by implementing the Uniswap V3 paper for the MVP. Then, we will consider changes to the existing logic.

Concentrated Liquidity Tasks

Preview Give feedback
  1. 4 of 4
    C:x/gamm F: concentrated-liquidity T:story
  2. 8 of 8
    C:x/gamm F: concentrated-liquidity T:story
  3. C:x/concentrated-liquidity F: concentrated-liquidity T:story
  4. F: concentrated-liquidity
    AlpinYukseloglu stackman27
  5. C:x/concentrated-liquidity F: concentrated-liquidity T:story
  6. C:x/swap-router F: concentrated-liquidity T:story
  7. F: concentrated-liquidity T:story T:tests T:tracking issue ⛰
  8. F: concentrated-liquidity T:story
    stackman27
  9. C:x/gamm F: concentrated-liquidity T:tracking issue ⛰
  10. 6 of 6
    F: concentrated-liquidity T:tracking issue ⛰
    p0mvn

Project View

For project view, all remaining issues and milestones, please see this board:
https://github.com/orgs/osmosis-labs/projects/1/views/69

For internal audit issues, please see this board:
https://github.com/orgs/osmosis-labs/projects/1/views/74

For OtterSec audit, use this:
https://github.com/orgs/osmosis-labs/projects/1/views/75

Scope of changes

Ordered by perceived risk:

  • x/concentrated-liquidity
    • The module with the core CL logic
    • Completely unreleased, needs everything reviewed
  • x/superfluid
    • Many changes related to introducing the new logic into the superfluid feature
    • Keep in mind slashing logic that happens in BeginBlock / EndBlock. Panicking here would cause chain halt.
  • x/incentives and x/pool-incentives
  • x/gamm
    • This is the module containing the balancer and stableswap pool management
    • We added new migration management functionality here
  • x/poolmanager
    • This is the module that manages pool creation, ID management and swap routing
    • It has been partially releases during the previous upgrade
    • A large portion of the changes is ported from pre-existing x/gamm module
    • Things to focus on (new logic):
  • x/twap
    • Changes and integrations of CL into x/twap
  • x/protorev
    • Module that does backrunning of swaps
    • Scope of changes: integrations into the CL module to grab highest liquidity CL pools for determining which pools to backrun
  • x/lockup
    • Keep in mind BeginBlock / EndBlock lock maturity. Panicking here would cause a chain halt.
    • Module that manages locking of assets
    • Some changes stemming from superfluid integratino
  • v16 upgrade handler logic
    • This logic runs during upgrade and creates a canonical pool from the pre-existing OSMO/DAI pool

Documentation

Up-to-date

Diagrams

Useful resources

Might be outdated

GitHub Issue Labels

Acceptance Criteria

  • concentrated liquidity MVP works
@osmo-bot osmo-bot moved this to Needs Review 🔍 in Osmosis Chain Development Oct 17, 2022
@p0mvn p0mvn added C:x/gamm Changes, features and bugs related to the gamm module. T:epic ⛰ labels Oct 17, 2022
@p0mvn p0mvn changed the title [Epic]: Concentrated Liquidity (WIP) [EPIC]: Concentrated Liquidity (WIP) Oct 17, 2022
@p0mvn p0mvn changed the title [EPIC]: Concentrated Liquidity (WIP) [EPIC][CL]: Concentrated Liquidity (WIP) Oct 17, 2022
@p0mvn p0mvn added the F: concentrated-liquidity Tracking the development of concentrated liquidity feature to improve filtering on the project board label Oct 19, 2022
@p0mvn p0mvn changed the title [EPIC][CL]: Concentrated Liquidity (WIP) [EPIC][CL]: Concentrated Liquidity Oct 19, 2022
@p0mvn p0mvn added this to the CL: Epic and User Stories milestone Jan 12, 2023
@daniel-farina daniel-farina removed this from the CL: Epic and User Stories milestone Mar 13, 2023
@p0mvn p0mvn removed their assignment Mar 22, 2023
@czarcas7ic
Copy link
Member

I am going to close this since we are now using the sprint board to track. Please feel free to re-open if you feel this provides some kind of clarity that the sprint board does not

@github-project-automation github-project-automation bot moved this from Needs Triage 🔍 to Done ✅ in Osmosis Chain Development Apr 26, 2023
@p0mvn
Copy link
Member Author

p0mvn commented May 4, 2023

I'm going to reopen this to keep the list of resources here for the auditors

@p0mvn p0mvn reopened this May 4, 2023
@github-project-automation github-project-automation bot moved this from Done ✅ to In Progress🏃 in Osmosis Chain Development May 4, 2023
@p0mvn p0mvn closed this as completed Aug 8, 2023
@github-project-automation github-project-automation bot moved this from In Progress🏃 to Done ✅ in Osmosis Chain Development Aug 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C:x/gamm Changes, features and bugs related to the gamm module. F: concentrated-liquidity Tracking the development of concentrated liquidity feature to improve filtering on the project board T:epic ⛰
Projects
Archived in project
Development

No branches or pull requests

3 participants