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

get_ticker returning "inf" for latest and percent_change #505

Closed
svk31 opened this issue Nov 26, 2017 · 4 comments
Closed

get_ticker returning "inf" for latest and percent_change #505

svk31 opened this issue Nov 26, 2017 · 4 comments
Labels
api bug
Milestone
201712 - Non-Cons...

Comments

@svk31
Copy link
Contributor

svk31 commented Nov 26, 2017

I'm working on replacing the old market stats calculations of the GUI with calls to get_ticker and ran into some cases where the API returns "inf" values.

image
@svk31 svk31 added the bug label Nov 26, 2017
@abitmore
Copy link
Member

@svk31 try develop branch of bitshares-core?

@abitmore
Copy link
Member

It should have been fixed in #455

@abitmore
Copy link
Member

By the way, this "inf" data is caused by a "nothing for something" scenario due to rounding issues.

On block 22127637, gold-star paid 0 BTC for 0.0004 CNY from x-bot (http://cryptofresh.com/tx/ae7f6770e96bf486c5381402e93d65c5729bde13).

Related issues: #132, #184.

@abitmore abitmore added this to the Next Non-Consensus-Changing Release - 201712 milestone Nov 27, 2017
@abitmore abitmore mentioned this issue Nov 27, 2017
Closed
@abitmore abitmore added the api label Nov 27, 2017
@abitmore
Copy link
Member

Should have been fixed. Closing. Feel free to reopen or open a new ticket if there are still issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
api bug
Projects
None yet
Milestone
201712 - Non-Consensus-Changing Release
Development

No branches or pull requests
2 participants
@svk31 @abitmore