Comments (3)
I suspect this the cause of your error. Can you confirm if you had BNB balance in your account at the time of the error?
Yep that was the cause of it. Brilliant, thanks.
from binance-oco.
@aadon94 Thanks for the feedback and for raising this issue.
The current behaviour is to reduce the sell amount by the fee paid, but only if the fee is not paid using BNB. This is to allow user who have selected to use BNB to pay fees (https://support.binance.com/hc/en-us/articles/115000583311) to trade the entire amount.
The logic is here:
Lines 161 to 166 in 3321bf4
However, this causes an issue when:
- BNB is the asset being purchased eg. when trading BNBBTC, BNBETH, BNBUSDT pairs, and
- there is insufficient BNB in the account to cover fees before the buy order is filled, meaning the fee is deducted from the buy amount
The reason is that orders for BNB pairs will always pay fees using BNB. As the script does not reduce the sell amount when fees are paid using BNB, there will be insufficient balance for the sell order if the account does not already have enough BNB to pays fees.
I suspect this the cause of your error. Can you confirm if you had BNB balance in your account at the time of the error?
from binance-oco.
@aadon94 This issue has been addressed in v1.5.0.
This release adds a -F
/--non-bnb-fees
option to force non-BNB fee calculation ie. the sell order amount will be reduced by 0.1%.
from binance-oco.
Related Issues (17)
- OCO Function HOT 6
- This is doing string comparisons, not numbers, for the prices HOT 1
- how to do this ? HOT 1
- waitForSellOrderFill(...).finally is not a function HOT 6
- What do you use this for? HOT 1
- Entry, stop and take profit and go sleep HOT 2
- key error HOT 7
- put a trade bellow using the same coin HOT 2
- it did not send the TP for the second trade HOT 2
- Order does not exist HOT 6
- Is this valid for Margin Trading? HOT 1
- Scaled Orders HOT 3
- Feature Request - Safe Exit to USDT after ALT/BTC trade HOT 1
- Feature Request - Set multiple profit target HOT 1
- Scale out logic broken in recent 2.0 update HOT 5
- 2.1.1 Limit sell order error HOT 2
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from binance-oco.