leaderboard updates for mitch #1275
Conversation
owocki
commented
May 25, 2018
owocki
commented
- adds the ability to filter on org name
- adds num bounties in addition to just the amount funded
ghost
assigned 
|
@owocki Thanks a lot! Please ping me here when this dude is merged, this will really help my workflow :) |
| @@ -164,6 +164,8 @@ class LeaderboardRank(SuperModel): | |||
| leaderboard = models.CharField(max_length=255) | |||
| amount = models.FloatField() | |||
| active = models.BooleanField() | |||
| count = models.IntegerField(default=0) | |||
There was a problem hiding this comment.
Do we want to denormalize the count field here? We could benefit from queryset caching and might not have a need for maintaining this field moving forward. How's the overhead for the operation to update this count/rank?
There was a problem hiding this comment.
this is where it's computed -> https://github.com/gitcoinco/web/pull/1275/files#diff-c672b9ab5fac2576200d174f3f05baf0R80
we'd have to change the way we store / compute the info in this PR to do this. not sure the performance implications, but right now the assemble_leaderboards script is fairly fast. its about 15s for the script to run once a day
|
@owocki Excellent, thanks a million! |
