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
@ned Ned looks 70 incorrectly.
@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.
@steemitdev/upcoming-changes-to-api-steemit-com