-
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
General improvements to gateway #5381
General improvements to gateway #5381
Conversation
…nce_manager (feat) Include nonce expiry in `NonceManager`
…e_client_order_tracker_v2
…at/gateway_improvement_part_1
@rapcmia The pr is ready again. Let me know if you have any other issues. |
PR update:
However when we are testing the trade execution for sushiswap-kovan, we are getting Client logs
Docker logs:
For Pangolin and TraderJoe, we are still getting the same behavior when checking balance and running the Amm-arb strategy
|
@james-hummingbot @vic-en Uniswap/Sushiswap kovan now fixed 👍 commit 61deddc Pangolin avalanche mainnet:
Actual: |
@nikspz I believe that issue is fixed. Can you test pangolin again? |
PR update:
However, upon checking the logs, it would return an error of
|
@vic-en Upon crosschecking issue still showing Steps:
Actual: my Env: AWS, Ubuntu Source |
You're getting this because the connection you added while testing uniswapLp connector/strategy persisted. |
@nikspz I believe some steps are missing from above. I doubt the client would be starting the custom build docker just with the steps you've given. However, I did further attempt to start a strategy with an account with less funds and definitely not blocked by the issue you're reporting.
|
…a/hummingbot into feat/gateway_improvement_part_1
@vic-en thanks for the info, there are no changes on the settings we have atm for We made sure as well the docker gateway image is created within the I did a clean uninstall/install and made sure no docker instance is present even on WindowsTerminal_eFot9Lro26.mp4 |
@vic-en the filtering is working, but |
…CoinAlpha/hummingbot into feat/gateway_improvement_part_1
@rapcmia Sorry for the misunderstanding. Turned out the tokens you were testing with weren't filtered. |
@vic-en Thanks for the update and no worries, we are seeing some changes on the latest commit. Test in progress |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Test Performed
- Installed branch successfully
- Generated certificate and gateway create with no issues
- Connected Uniswap, Susushiswap, Pangolin Traderjoe and checked balance with no issues
- Created simple AMM_Arbitrage and started strategy
- Verified no arbitrage opportunity in logs
- Verified filled orders and data match with history
- Confirmed CSV file and txhash to match on snowtrace
- Tested on Source AWS Instance and Docker Build
Before submitting this PR, please make sure:
A description of the changes proposed in the pull request:
GatewayInFlightOrder
to useInFlightOrder
gateway status
/network/status
callsgateway create
logic when infura api key is addedTests performed by the developer:
Tips for QA testing:
Try running hummingbot and gateway with various amm_arb strategies and exchanges.