Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Part of accumulated qty is missing if the qty has more decimal places than min lot size of CEX #40

Open
raladev opened this issue Jul 6, 2020 · 1 comment
Labels
bug Something isn't working P3 low priority tbd to be discussed

Comments

@raladev
Copy link
Contributor

raladev commented Jul 6, 2020

Steps:

  1. CEX - Binance, min lot size - 0.005 ETH
  2. Fill 0.0005 qty -> Accumulated qty is 0.0005
  3. Fill 0.005 qty

Actual result:

  1. 0.0055 qty order is sent to CEX -> 0.005 qty is filled
  2. Accumulated qty is 0

Expected result:

  1. 0.005 qty order is sent to CEX -> 0.005 qty is filled
  2. Accumulated qty is 0.005

Screenshot from 2020-07-06 14-12-41

@raladev raladev added bug Something isn't working P3 low priority labels Jul 6, 2020
@kilrau kilrau assigned ghost Jul 6, 2020
@ghost
Copy link

ghost commented Jul 31, 2020

I think the nominal $ amount is so small 0.0005 ETH = ~$0.16 that this is not worth spending time on.

@ghost ghost added tbd to be discussed and removed P3 low priority labels Jul 31, 2020
@kilrau kilrau added the P3 low priority label Jul 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working P3 low priority tbd to be discussed
Projects
None yet
Development

No branches or pull requests

2 participants