search

CyberPunkMetalHead / Binance-News-Sentiment-Bot

This is a fully functioning Binance trading bot that takes into account the news sentiment for the top 100 crypto feeds. If you like this project consider donating though the Brave browser to allow me to continuously improve the script.
MIT License
1.57k stars 290 forks source link

'could not parse' error #32

Closed simin75simin closed 3 years ago

simin75simin commented 3 years ago

an example would be: Could not parse https://www.coin.space/feed/ error is: this happened on google colab, and my own machine (ubuntu 18.04, python3.7.10). most of the news source it cannot parse at this moment, which is kinda worrying. thanks.

CyberPunkMetalHead commented 3 years ago

Can you increase the time to 60 seconds in line 220 and try again? async with session.get(feed, headers=headers, timeout=7) as response:

OdinLin commented 3 years ago

session.get(feed, headers=headers, timeout=7, verify_ssl=False) as response work for me

simin75simin commented 3 years ago

thanks for the potential fixes. using google colab (i need to use a proxy for binance and everything but binance's ordering system i.e. sockets apparently does not hook up with the proxy i set up when i initialize the client. see: https://github.com/sammchardy/python-binance/issues/248#issuecomment-451898555) here's what i got: while none of those fixes gets me through every site, i find a combination of them work better, leaving out only six sites or so. what the program reads affects its trading behavior. every fix alone works better than no fix.