CyberPunkMetalHead / gateio-crypto-trading-bot-binance-announcements-new-coins

This is a crypto trading bot that scans the Binance Annoucements page for new coins, and places trades on Gateio
MIT License
1.22k stars 302 forks source link

Not buying / not spotting new listings #78

Closed alecsiemerink closed 1 year ago

alecsiemerink commented 2 years ago

The bot does not seem to be buying anything / spotting newly listed coins. In previous versions it did buy, but couldnt sell due to 'not enough balance' errors.

There isn't anything to be seen in the logs.

Does anyone have a fix for this problem?

oonyeje-bside commented 2 years ago

Also seeing the same issue

DominicFrei commented 2 years ago

@alecsiemerink @oonyeje-bside Does the coin itself get detected or this where the problem is? Can you two post your logs just so that everyone can have a look at it? Also, which version of the bot did you use?

oonyeje-bside commented 2 years ago

@DominicFrei

2021-11-17 13:14:42,044 INFO: No coins announced, or coin has already been bought/sold. Checking more frequently in case TP and SL need updating
2021-11-17 13:14:44,463 INFO: Checking for coin announcements every 3 seconds (in a separate thread)
2021-11-17 13:14:45,054 INFO: No coins announced, or coin has already been bought/sold. Checking more frequently in case TP and SL need updating
2021-11-17 13:14:48,064 INFO: No coins announced, or coin has already been bought/sold. Checking more frequently in case TP and SL need updating
2021-11-17 13:14:48,415 INFO: Checking for coin announcements every 3 seconds (in a separate thread)
2021-11-17 13:14:51,068 INFO: No coins announced, or coin has already been bought/sold. Checking more frequently in case TP and SL need updating
2021-11-17 13:14:52,418 INFO: Checking for coin announcements every 3 seconds (in a separate thread)
2021-11-17 13:14:54,085 INFO: No coins announced, or coin has already been bought/sold. Checking more frequently in case TP and SL need updating
2021-11-17 13:14:56,367 INFO: Checking for coin announcements every 3 seconds (in a separate thread)
2021-11-17 13:14:57,087 INFO: No coins announced, or coin has already been bought/sold. Checking more frequently in case TP and SL need updating
2021-11-17 13:15:00,094 INFO: No coins announced, or coin has already been bought/sold. Checking more frequently in case TP and SL need updating
2021-11-17 13:15:00,309 INFO: Checking for coin announcements every 3 seconds (in a separate thread)
2021-11-17 13:15:03,099 INFO: No coins announced, or coin has already been bought/sold. Checking more frequently in case TP and SL need updating
2021-11-17 13:15:04,303 INFO: Checking for coin announcements every 3 seconds (in a separate thread)
2021-11-17 13:15:06,108 INFO: No coins announced, or coin has already been bought/sold. Checking more frequently in case TP and SL need updating
2021-11-17 13:15:08,252 INFO: Checking for coin announcements every 3 seconds (in a separate thread)
2021-11-17 13:15:09,111 INFO: No coins announced, or coin has already been bought/sold. Checking more frequently in case TP and SL need updating

it basically repeats that and I forked off the latest master 2 days ago

tejado commented 2 years ago

It does not detect the coin in the latest listings as it does not match the regex. It matches only for coins in brackets, e.g. "(MOVR)" but not MOVR or not MOVR/USDT. We have to adjust it.

So currently, the regex is the following: found_coin = re.findall('\(([^)]+)', latest_announcement)

It would be good to check as well for anything uppercase in front of e.g. /USDT (or /[A-Z]+) and anything uppercase surrounded by spaces.

DominicFrei commented 2 years ago

It does not detect the coin in the latest listings as it does not match the regex. It matches only for coins in brackets, e.g. "(MOVR)" but not MOVR or not MOVR/USDT. We have to adjust it.

So currently, the regex is the following: found_coin = re.findall('\(([^)]+)', latest_announcement)

It would be good to check as well for anything uppercase in front of e.g. /USDT (or /[A-Z]+) and anything uppercase surrounded by spaces.

@tejado New pairs like MOVR/USDT are left out on purpose because they do not lead to a reliable pump. Can you post a specific announcements where the initial(!) announcement happened without brackets?

tejado commented 2 years ago

@DominicFrei I guess it was not clear, that these are left out on purpose. Do you have any examples where it did not lead to a pump?

DominicFrei commented 2 years ago

@tejado I remember looking into this a while ago and couldn't find a reliable pattern. Can you post some examples (more than just one since we need a pattern)? That would be great!

alecsiemerink commented 2 years ago

@DominicFrei New Listings are getting detected, but no orders are being placed. Also logs: uppers=None since last commit.

Couldn't find why it's been doing this.