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.

Sign up for GitHub

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

Jump to bottom

user added liquidity and avg price calc is incorrect #10510

Open
bthaile opened this issue Feb 3, 2021 · 0 comments
Open

user added liquidity and avg price calc is incorrect #10510

bthaile opened this issue Feb 3, 2021 · 0 comments
Assignees
@bthaile
Labels
prio/1 High
Projects
Augur Simplified

Comments

@bthaile
Copy link
Contributor

bthaile commented Feb 3, 2021

added liquidity to existing AMM Yes/NO .33/.67

999659999 / 507463 = 1969.917016610078

cashValue / netShares should give avg price, I would expect the user's "No" Shares to be avg price of 0.67 but got 0.20 (after rounding up from .1969

image
@bthaile bthaile added the prio/1 High label Feb 3, 2021
@bthaile bthaile added this to To Do in Augur Simplified via automation Feb 3, 2021
@bthaile bthaile assigned bthaile and unassigned justinbarry Feb 3, 2021
@bthaile bthaile mentioned this issue Feb 3, 2021
Merged
@bthaile bthaile moved this from To Do to Done in Augur Simplified Feb 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bthaile bthaile

Labels
prio/1 High
Projects
Augur Simplified
  
Done
Milestone
No milestone
Development

No branches or pull requests
2 participants
@justinbarry @bthaile