Skip to content
This repository has been archived by the owner on Mar 25, 2024. It is now read-only.

When streaming trade activity, the profitMinusBags value is skewed #16

Open
victorbutler opened this issue Apr 22, 2018 · 3 comments
Open
Labels
bug Something isn't working testing Currently testing this issue
Milestone

Comments

@victorbutler
Copy link
Owner

profitPlusBags is fine

Maybe it has something to do with the order the pair data is entered into the history?

@victorbutler victorbutler added the bug Something isn't working label Apr 22, 2018
@victorbutler victorbutler added this to the v0.1 milestone Apr 22, 2018
@victorbutler
Copy link
Owner Author

Might have fixed this with commit 30f7f84

@victorbutler victorbutler added the testing Currently testing this issue label Apr 26, 2018
@victorbutler
Copy link
Owner Author

Basic trade activity testing is OK. Waiting to test more complicated trade activity.

@victorbutler
Copy link
Owner Author

Partial fill execution skews the profit - bags. Something's going on with the bag resolution code. Look inside the code that tries to zero out the Quantity. A zero Quantity should remove that bag since buys and sells have been fully balanced out.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working testing Currently testing this issue
Projects
None yet
Development

No branches or pull requests

1 participant