-
Notifications
You must be signed in to change notification settings - Fork 7
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
SortBy-Hot doesn't work #136
Comments
Note the time stamps. |
We need to test "Hot" sort better - if it doesn't work at all - something should be done. |
Here's where the ranking is set: https://github.com/TheRealGD/therealgd/blob/develop/src/Entity/Submission.php#L327-L340 Almost any time the submission is updated, they had the entity recalculate it's ranking score so that the submissions can be indexed by rankings. Unfortunately, it means migrating every submission when tweaking the ranking algorithm. Also, here's where the sorting actually happens in the SubmissionRepository: https://github.com/TheRealGD/therealgd/blob/develop/src/Repository/SubmissionRepository.php#L180-L183 |
Please see issue #148 I created earlier today regarding ranking algorithms. @DEVxALMIGHTY and I can probably help with this, should we need to make it a v1 priority. |
I prefer New, but I'm pretty sure Hot is critical. |
This is in progress now on triage board |
For the time being, let's replace the existing "hot" formula with the xkcd formula:
I don't know how to PHP, so whoever can implement ¯\_(ツ)_/¯ |
There's also something done in #200 IIRC on this. Should look at both before implementing |
Semper's Harbor Freight post had -2 and I posted a new deal, yet his was still placed above mine.
The text was updated successfully, but these errors were encountered: