search

tudorelu / pyjuque

⚡ Open Source Algorithmic Trading Bot for Python.
MIT License
456 stars 90 forks source link

<class 'ccxt.base.errors.Order Not Found'> in updateOpenOrder() #17

Open schykowski opened 3 years ago

schykowski commented 3 years ago

When having more then one pair in open Sell orders recive Order Not Found from Binance code: -2013, msg: Order does not exists.

Issue is only on testing False condition (works great in test mode) and only if more then one pair is in sell ored pool.

schykowski commented 3 years ago

One more from my observations, sometimes two or three pairs not rise this error. If no other pair has a change in the buy/sell status. If all entry signal was not changed before exit signal the error is not showing. Hope U can understand what i want to say, my English is not very well. :)

tudorelu commented 3 years ago

Hey man, thanks for submitting an issue!

Could you please paste as much of the code you are running as possible? If you do it I will try to recreate your issue so I can debug it.

tudorelu commented 3 years ago

When having more then one pair in open Sell orders recive Order Not Found from Binance code: -2013, msg: Order does not exists.

Issue is only on testing False condition (works great in test mode) and only if more then one pair is in sell ored pool.

So This is happening when test_mode = False. OK. I need a minimum amount of code that can recreate your error.

schykowski commented 3 years ago

Changes witch i made in Your files is: adding a apprise as notification module google sheet integration fix check the price in bot_Control ( was > 0 and now is > current_price) change the time limit for open buy order in bot_control change stop_loss rule in bot_control added notification function in bot_control

schykowski commented 3 years ago

and last word :) i am a beginner with programming so probably it is not an perfect example how this should be done, but still learning :) BTW is the any way to get a pair name and buy price from strategy class? I try to create a log to file but without this i am naming the files and notification by time not by pair, second will be goo to check the buy price to decide want to risk more or not :)

tudorelu commented 3 years ago

Hey man, no offense. I would prefer not to open zip files.

If you can recreate a minimal code example that shows the behavior you're encountering, I'm happy to help. But I see you've been updating the pyjuque 'core' code, so that bug might have been introduced by you.

Alternatively, make a fork of this repo, add your updates and then make a pull request, which I will be happy to check.

schykowski commented 3 years ago

sure i will prepare all in some logical way and paste this here.