-
-
Notifications
You must be signed in to change notification settings - Fork 33
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
GIVpower Boosting | Make projected rank update dynamically #1904
Comments
Please check this scenario @MohammadPCh @aminlatifi But I expected that the current project, which has a rank 22(Cascades Raptor Center) and the following ranks will decrease by one rank(22->23/23->24/....) But some like 23 and 24 are calculated correctly 1- I think there is a problem here that the expected rank for all projects is not calculated correctly |
3- I think we have another problem because for this project (and some project else) https://giveconomy.vercel.app/project/greater-vancouver-food-bank |
It's normal 😄
Probably the project didn't have that power (corresponding to rank 22) for all the previous round! |
Nice catch @maryjaf |
@laurenluz @maryjaf @aminlatifi I increased the accuracy of decimal numbers. So it must be fixed. |
No, it didn't solve the issue. We must solve it fundamentally! |
I made these PRs The backend has issues with tests, it's late here maybe Carlos can found the issue and solve them. If that goes through the frontend PR will be ready for review |
Backend PR is merged, the fronend must work fine. But I made a PR to improve it |
Please mention me when the changes are ready to retest on Giveconomy barnch @aminlatifi @MohammadPCh @alireza-sharifpour |
The Amin's PR on the Frontend is merged and it's on Giveconomy branch. @maryjaf |
Two projects are shown as #11 ranked @aminlatifi @alireza-sharifpour |
@maryjaf Are their total power different? |
No, it is the same for both |
@maryjaf Yes, they must have the same rank. In backend we limited the power precision to 2 fraction digits |
Thanks @aminlatifi |
If their future total power is more than or equal power of the 21st project and less than the 20th project, that value is correct. @maryjaf Is it the case/ |
Total Givpower of "Jewish Federation of Northern New Jersey, Inc" was 8084 Giv and "Cascades Raptor Center" was 6206 Giv but for both of them the projected rank = 21 |
According to the conversation we had with @aminlatifi , with the new changes, it has been accepted that the calculated projected rank is not necessarily correct @laurenluz |
From GIVeconomy call, we want to have projected rank show impact immediately (or as fast as possible).
Planning to make a hack in the frontend & build and API.
@MohammadPCh & @aminlatifi have a plan. Notes from GIVeconomy call are here
cc @MoeNick @maryjaf
The text was updated successfully, but these errors were encountered: