-
Notifications
You must be signed in to change notification settings - Fork 42
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
关于调整 Wtfswap 流动性区间逻辑的方案 #94
Comments
@hattiepwvi @gpteth 也可以看看,这样 Pool 合约的实现应该会更简单一些。 |
Open
好的,收到, 我仔细学习看看
愚指导 ***@***.***> 于2024年6月18日周二 12:15写道:
… @hattiepwvi <https://github.com/hattiepwvi> @gpteth
<https://github.com/gpteth> 也可以看看,这样 Pool 合约的实现应该会更简单一些。
—
Reply to this email directly, view it on GitHub
<#94 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AYSDDWW46YFROYDDTL3TPDDZH6X43AVCNFSM6AAAAABJPGJRD6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNZUHE2TIMJQGI>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Open
Open
已经调整 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
和 @Web3Pignard 讨论了一下,当前 positionType 分别取 -1, 0, 1 ,代表低于价格范围下限、在价格范围内、超出价格范围,这个逻辑可能存在一个问题。
价格范围是在 Pool 的 initialize 方法中初始化的,而这个方法只能调用一次,这样会导致 Pool 的价格范围可能会被恶意设置,或者会由于代币价格的波动导致价格的范围变得没有意义。
Uniswap V3 的价格范围是设置在 Position 上面的,但是这样会导致收取 fee 的逻辑太复杂,不利于教学。经过讨论后决定基于 Uniswap V2 的逻辑进行改进,吸收 Uniswap 价格区间的概念。不同的价格区间都可以创建出不同的 Pool,而不是像 V2 那样默认是 0 到正无穷。另外我们处于教学的目的,不用像 V3 那样考虑充分利用流动性,所以不用基于 Position 来设置价格范围。只需要支持在一个池子内交易即可。
修改后的逻辑变成:
The text was updated successfully, but these errors were encountered: