Skip to content
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

Improve distance leaderboard algorithm #106

Open
diogogmatos opened this issue Sep 5, 2024 · 0 comments · May be fixed by #115
Open

Improve distance leaderboard algorithm #106

diogogmatos opened this issue Sep 5, 2024 · 0 comments · May be fixed by #115
Assignees
Labels
enhancement New feature or request

Comments

@diogogmatos
Copy link
Member

diogogmatos commented Sep 5, 2024

Currently, the distance leaderboard calculates the score of a user by doing the sum of distances from the CeSIUM "headquarters" to their pins.

This means that the more pins someone places close together to each other, the higher their score will be, which is unfair since we want to value distance traveled to place a pin and not quantity of pins.

Some better alternatives would be:

  • Group pins from the same person that are placed close together, calculating the average distance of the pins to the headquarters.
  • Same as the previous solution, but instead choose the highest distance from the group of pins.
  • Same as the previous solution, but instead choose the lowest distance from the group of pins.

To group the pins, there could be two alternatives:

  • Create the concept of "trip" and make users manually mark a group of pins as belonging to the same trip on the submission process. Pins from the same trip would be grouped and count as a single distance.
  • Mathematically try to group pins from the same user by checking, for each pin, if there are others close by (the radius could be specified to a specific distance).
@diogogmatos diogogmatos added the enhancement New feature or request label Sep 5, 2024
@AfonsoMartins26 AfonsoMartins26 self-assigned this Sep 16, 2024
@AfonsoMartins26 AfonsoMartins26 linked a pull request Sep 28, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants