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

Are the parameters to computeProfitMaximizingTrade correct? #24

Open
tthil opened this issue Sep 26, 2022 · 5 comments
Open

Are the parameters to computeProfitMaximizingTrade correct? #24

tthil opened this issue Sep 26, 2022 · 5 comments

Comments

@tthil
Copy link

tthil commented Sep 26, 2022

The call to the smart contract function computeProfitMaximizingTrade takes 4 parameters (sReserve0, sReserve1, uReserve0, uReserve1)
The parameter list in the smart contract is defined as
computeProfitMaximizingTrade(
uint256 truePriceTokenA,
uint256 truePriceTokenB,
uint256 reserveA,
uint256 reserveB

My question is if sReserve0 and sReserve1 really are truePriceTokenA and truePriceTokenB?

@0xNagato
Copy link

Were you able to improve this?

@tthil
Copy link
Author

tthil commented Nov 17, 2022 via email

@0xNagato
Copy link

Did you have any luck finding any good ones? I'm very stuck

@tthil
Copy link
Author

tthil commented Nov 17, 2022 via email

@manzub
Copy link

manzub commented Nov 12, 2023

Hi Thomas, I do want to build it myself too but i need inspiration to build a more competitive bot joining this late

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

No branches or pull requests

3 participants