[busy][2.5.6] inconsistent reputation score
Project Information
- Repository: https://github.com/busyorg/busy
- Project Name: Busy
I reported two issues on the reputation score, and made the fix for the first issue, which is now merged. I also explained the cause of the second issue.
- Issue:
- PR: https://github.com/busyorg/busy/pull/2146 (merged here)
Expected behavior
- Reputation score should be floored.
- Reputation score should be consistent
Actual behavior
- Reputation score is rounded
- Reputation scores in user info and post are different.
How to reproduce
floor vs round
Pick any user whose reputation score's first digit after the decimal point is greater than 5. Then the reputation score on steemit.com and busy.org are different.
inconsistency between user info and post
This occurs for most users, but the difference is noticeable for relatively low reputation users since normalized reputation can be more different.
- Browser version: Any
- App version: Busy 2.5.6 (current)
- Operating system: Any
Recording Of The Bug
floor vs round
https://busy.org/@ned Ned looks 70 incorrectly.
https://steemit.com/@ned Nop, Ned is not 70 yet.
inconsistency between user info and post
User info says it's 65.024
But, post says it's 65.019
Cause and Fix
floor vs round
This is due to incorrect math function (round instead of floor) and I made the fix: https://github.com/busyorg/busy/pull/2146 (merged here)
inconsistency between user info and post
This is due to the deprecated API get_state explained here:
If any apps are still relying on get_state, we highly recommend ceasing use as soon as possible.
https://steemit.com/steemit/@steemitdev/upcoming-changes-to-api-steemit-com
Thanks for your contribution.
Great to see a fix for the issue and great to see it merged. Although I know the reputation could change following your post, it would have great for you to provide a few examples rather than stating pick any user, it's overall makes it a little easier to understand and overall helps to the contribution to flow a little better. A small improvement and you did a nice job showing in your images to provide more clarity.
Keep up the good work and look forward to more contributions from you.
Your contribution has been evaluated according to Utopian policies and guidelines, as well as a predefined set of questions pertaining to the category.
To view those questions and the relevant answers related to your post, click here.
Need help? Chat with us on Discord.
[utopian-moderator]
Thank you for your review, @tobias-g! Keep up the good work!
Summary in Korean: 마침 바로전 유토피안 글로 명성도 65가 된 기념으로 명성도 관련 버그 유토피안 글을 씁니다^^ 일반 글을 좀 쓰고 싶은데 계속 출장중이어서ㅠㅠ 그냥 상당부분 이전에 준비된 단순한 유토피안글만 계속 쓰고 있습니다ㅠㅠ 신경안쓰셔도 될만한 사소한 버그 수정과 리포트입니다. 스팀잇과 동일하게 하려면 명성도는 반올림이 아닌 버림을 해야하고(이건 오래전에 고쳐둔 것) 그리고 hivemind도입에 따른 api변경으로 유저정보에 보이는 명성도와 포스팅 옆에 나오는 명성도가 다른 버그가 생기네요. 이건 busy의 경우 아주 간단히 수정가능할 것 같지는 않아서 리포트만 했습니다. 아무튼 busy개발자가 드디어 다시 일을 하기 시작했네요ㅎㅎ 몇몇 간단한 수정사항이 조만간 반영될지도.
busy 개발자가 드디어 일을 하기 시작했군요. 그리고 명성 65 축하합니다. ㅎㅎ
감사합니다^^ relative url도 merge가 됐으니 곧 deploy가 되겠네요.
짱짱맨 호출에 응답하여 보팅하였습니다.
짱짱맨 보팅이 누적 3000 SBD를 돌파하였습니다. 참여와 응원에 감사드립니다.
짱짱맨 누적보팅 3000SBD 돌파
Hi @blockchainstudio!
Your post was upvoted by @steem-ua, new Steem dApp, using UserAuthority for algorithmic post curation!
Your post is eligible for our upvote, thanks to our collaboration with @utopian-io!
Feel free to join our @steem-ua Discord server
Hey, @blockchainstudio!
Thanks for contributing on Utopian.
We’re already looking forward to your next contribution!
Get higher incentives and support Utopian.io!
Simply set @utopian.pay as a 5% (or higher) payout beneficiary on your contribution post (via SteemPlus or Steeditor).
Want to chat? Join us on Discord https://discord.gg/h52nFrV.
Vote for Utopian Witness!