Skip to content

Integrate Liquidity Provider feature#2939

Merged
basfroman merged 51 commits intostagingfrom
feat/roman/liquidity-provider
Jul 3, 2025
Merged

Integrate Liquidity Provider feature#2939
basfroman merged 51 commits intostagingfrom
feat/roman/liquidity-provider

Conversation

@basfroman
Copy link
Collaborator

@basfroman basfroman commented Jun 27, 2025

Content

Extrinsics:

  • add_liquidity_extrinsic
  • modify_liquidity_extrinsic
  • remove_liquidity_extrinsic
  • toggle_user_liquidity_extrinsic

Subtensor methods:

  • add_liquidity
  • modify_liquidity
  • remove_liquidity
  • toggle_user_liquidity
  • get_liquidity_list

Helpers

  • bittensor.utils.liquidity

This docement describes the mathematical functions used for calculations.

AC:

  • liquidity logic
  • unit tests:
    • subtensor
    • async subtensor
    • extrinsic module
    • async extrinsic mudule
    • liquidity utils
  • e2e tests (also tested locally with non-fast-blocks node)
  • ready to merge - subtensor PR is merged to devnet-ready
  • ready to release - subtensor PR is deployed to finney

@basfroman basfroman requested review from a team and gztensor July 1, 2025 19:39
@basfroman basfroman changed the title [WIP] Integrate Liquidity Provider feature Integrate Liquidity Provider feature Jul 1, 2025
Copy link
Contributor

@thewhaleking thewhaleking left a comment

Choose a reason for hiding this comment

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

Overall good. Just a few small items.

basfroman and others added 4 commits July 2, 2025 09:27
Co-authored-by: BD Himes <37844818+thewhaleking@users.noreply.github.com>
I have an entry for SDK v.10 to make error messages more informative. I don't want the user to have to double-check themselves to figure out what the error is.

Co-authored-by: BD Himes <37844818+thewhaleking@users.noreply.github.com>
@basfroman basfroman merged commit bcf4cc2 into staging Jul 3, 2025
135 checks passed
@basfroman basfroman deleted the feat/roman/liquidity-provider branch July 3, 2025 16:51
@ibraheem-abe ibraheem-abe mentioned this pull request Jul 7, 2025
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.

2 participants