Joystream / Joystream/joystream

AMM Design Discussion

Open
#3,754 12 comments 0 reactions 2 assignees View on GitHub

@dobertRowneySr is already working on this.

Since May 16, 2022.

project_token runtime
Dominant language
Rust
Stars
1.4k
Forks
116
PR merge metrics
No merged PRs in 30d

Description

Background

Great work with this overview @ignazio-bovo

https://hackmd.io/@msgSgRNqSKmkYgN4EoybcA/BysDdI8Iq

First thoughts

  1. It seems to me the best tradeoff here would be something that never is really bad, but works Ok in a wide range of circumstances, which would be Linear? It's also way easier for people to understand.
  2. When it says that Initial Liquidity bootstrapping capabilities are very good, does this not mean that its simply really cheaply selling off the tokens to buyers, which is at the expense of the issuer?
  3. I think we should just pick a single type of curve for our first implementation, adding additional curves in the future will be very easy, no migration needed, and it will be with the benefit of more concrete information.
  4. Ok, so a tax on both AMM transaction types.
  5. Price based slippage constraint seems most direct and simple, but is it the same as is used on say Uniswap? may be wise to just stick with their exact same version of slippage tolerance, as people already know this.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.