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

active_votes: inconsistent rep and missing time #161

Closed
crokkon opened this issue Dec 21, 2018 · 3 comments · Fixed by #225
Closed

active_votes: inconsistent rep and missing time #161

crokkon opened this issue Dec 21, 2018 · 3 comments · Fixed by #225
Labels

Comments

@crokkon
Copy link

crokkon commented Dec 21, 2018

I guess this was intended as a feature, but is there any chance to go back to providing the raw author reputation values everywhere instead of the normalized/log10 values?

steemit node:

curl -s --data '{"method": "tags_api.get_discussion", "params": {"author": "steemitboard", "permlink": "steemitboard-notify-stmdev-20181024t002841000z"}, "jsonrpc": "2.0", "id": 2}' https://api.steemit.com
[...] "active_votes":[{"voter":"stmdev","reputation":"59.5","percent":"10000","rshares":"0"}],"author_reputation":20209499381910, [...]
  • reputation in active_votes is the log10 value
  • author_reputation is the raw value
  • voting time and weight is completely missing

minnowsupport node:

curl -s --data '{"method": "tags_api.get_discussion", "params": {"author": "steemitboard", "permlink": "steemitboard-notify-stmdev-20181024t002841000z"}, "jsonrpc": "2.0", "id": 2}' https://steemd.minnowsupportproject.org
[...],"active_votes":[{"voter":"stmdev","weight":0,"rshares":0,"percent":10000,"reputation":"7203258450054","time":"2018-11-11T10:33:09"}],"replies":[],"author_reputation":"23856299575939"[...]

(reputation in active_votes is the raw value and author_reputation is the raw value, voting time is available).

Providing the log10 values at some places makes it much harder to write code that works with any node and the single-digit precision makes it harder to see the reputation difference between voter and author relevant for flags.

@roadscape
Copy link
Contributor

These condenser_api calls are supposed to convert rep back to raw even though it's normalized internally. This is a bug and needs to be fixed.

As for voting time and weight -- these contribute to an already huge response (for both post lists and threads). Generally the backend needs to move towards more lightweight responses, and the client should make explicit calls (e.g. get_active_votes) for detailed data when needed.

@roadscape roadscape changed the title Inconsistent reputation representation and missing time for active_votes active_votes: inconsistent rep and missing time Jan 17, 2019
@roadscape roadscape added the bug label Jan 17, 2019
@emre
Copy link

emre commented Jan 18, 2019

Seems like the raw reputation is explicitly set to a normalized value while indexing.

https://github.com/steemit/hivemind/blob/master/hive/utils/post.py#L116.

@roadscape
Copy link
Contributor

PR to fix returned reputation: #225

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants