-
Notifications
You must be signed in to change notification settings - Fork 2k
Error Apparently due to XMPP being enabled. #459
Comments
What is the command you are trying to run? |
All of the following commands produce the same error zenbot trade --conf ./livepnex-conf.js --period 1h poloniex.GNT-BTC zenbot trade --conf ./livepnex-conf.js --period 1h zenbot trade --conf ./livepnex-conf.js |
Warning: this is an ugly hack and I dont recommend this as a likely fix until a dev says something, but.. I was able to get trading to work by adding the circular-json library with npm: npm install --save circular-json then in zenbot/node_modules/sosa_mongo/mongo_backend.js adding this to line number 2 of mongo_backend.js I know this is probably not the best practice. |
I noticed some new output above the trading line. Maybe this will help diagnose something, it appears to not be saving the session from what I can tell. Any of the values below with *'s have been removed for my own privacy. ---------------------------- STARTING LIVE TRADING ---------------------------- 2017-08-10 01:46:43 - error saving session And then after this it begins live trading. |
After disabling xmpp, the error when starting trading went away. Edit: I Would like to be able to use XMPP for trading notification, so do you want me to open a new report? or can we turn this from a question into a bug? |
I have narrowed it down to enabling xmpp. When I enable xmpp. I get errors on just about every command about circular structure errors. When I disable xmpp. it works fine. I'd like to use xmpp. (Edit: it is worth noting, that with my hack allowing the circular structure to be flattened so it didnt puke on the trade command, the bot did sign on to xmpp, I am not 100% certain a trade occured during the time it was enabled, but I did see the account log on, I just never received any messages) |
It would seem to me that the xmpp object isnt being handled correctly (because it possibly contains a circular reference?) when it comes to the conf object for saving. Im not 100% sure as js is not one of the languages I frequently use. |
@dmzpkts |
Did you do an: npm install simple-xmpp I forgot to include this dependance in package.json |
I get this same issue, but unrelated to XMPP. I don't have XMPP enabled, but see this circular issue with the GDAX exchange in the trade file.
So I don't think this is XMPP related. Every session save fails from this point forward. I can dump the log output like @dmzpkts did if anyone wants to see it. I'm not fully sure how this will affect the bot going forward as it doesn't appear to be saving historical data to my DB so back analytics probably won't survive a restart? My thought is maybe I have corrupted data in Mongo? I think I may git bisect back out to an earlier checkout and see if the problem persists. |
As a follow-up, I was previously on commit 2bda376 and did a full git pull of So I rolled back to 2bda376, blew away That, I believe is related to the upgrade of other server software are my machine and potentially unrelated to zenbot. These errrors are:
|
I'm having the exact same thing as cpjolicoeur and I believe it's also due to a recent update. I have another zenbot that is working fine (no circular structure errors) and the only difference is that it was built two days ago. How do I roll back the version so I can get this working - thanks. |
I was running normally too - until today... same errors as above... |
I have managed to get zenbot to do everything but trade.
Upon attempting to trade I get this, right after the candlesticks.
(Edit: the reason the other commands were working was because I was using a conf file that did not have xmpp enabled, for testing, the only difference between the conf file I was using, and the live trading conf file was xmpp, and exchange keys.)
The text was updated successfully, but these errors were encountered: