-
-
Notifications
You must be signed in to change notification settings - Fork 7
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
Issue with ELF / POLY listing #4
Comments
I am also getting the same error. |
Sorry for the delay on this. I unfortunately don't have enough information to replicate the issue (I missed this sale myself) - I'm assuming there's a problem with the ticker format passed to the Binance API. If anyone has a record of the value of the ticker processed and sent to the API that would help. To avoid this issue in the future, I've added a verbose logging file that saves much more information about each attempted transaction. Please pull this new version and include the relevant information for this new log file for any other issues. I've also added 'retries' for connection refusals which periodically occur. |
Thanks, I think you are right as original repository has very simillar issue reported as well. There is json format mismatch for real trades. I will pull latest changes and keep it running and I will get back to you in case of any findings. |
I just reread my code again but still haven't found anything standing out as problematic. This script is executing quickly, I suppose it's possible the program is requesting a price from Binance before their API is even ready for it... I just added retry's to |
It looks like there was a new coin on Sept 22, did either of you happen to catch it in your logs @bwasik @easyguyme? I'm testing a new version with notifications (dev branch) and mistakenly missed it 😔. The verbose log should have enough information for me to fix whatever this issue is. |
I have no logs logged for sept 22 only 19th |
I missed two coins today, both with the "binance.exceptions.BinanceAPIException: APIError(code=-1121): Invalid symbol." Verbose logs were:
Actual error:
|
Fixed! Thank you @jarrodjay for posting. FYI, this new update also now has notifications - details are in the readme. Right now I have Discord and Telegram set up, but if anyone wants another service open an issue/discussion. |
Thank YOU for this fork. It's inspired me to start playing around with Python too. Love the notification setup. |
It didn't purchase and crashed with this log:
The text was updated successfully, but these errors were encountered: