-
Notifications
You must be signed in to change notification settings - Fork 266
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
Comments
Were you able to improve this? |
Hi,
No, I moved on. I felt like this repo is dead and more of a test.
//Thomas
… On 17 Nov 2022, at 00:53, 0XFEZ ***@***.***> wrote:
Were you able to improve this?
—
Reply to this email directly, view it on GitHub <#24 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAPF26ZPJQJF7BA63BSRBBTWIUNSJANCNFSM6AAAAAAQV6VQJQ>.
You are receiving this because you authored the thread.
|
Did you have any luck finding any good ones? I'm very stuck |
I decided to dig deeper into by myself and build it from scratch as it is more to it than just the interaction with the smart contract.
//Thomas
… On 17 Nov 2022, at 10:57, 0XFEZ ***@***.***> wrote:
Did you have any luck finding any good ones? I'm very stuck
—
Reply to this email directly, view it on GitHub <#24 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAPF265NXNDUBFILXCO6KGLWIWUJJANCNFSM6AAAAAAQV6VQJQ>.
You are receiving this because you authored the thread.
|
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
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?
The text was updated successfully, but these errors were encountered: