djcunningham0 / multielo

MIT License
62 stars 8 forks source link

You DO handle ties now :) #14

Open fab0i opened 2 years ago

fab0i commented 2 years ago

https://github.com/djcunningham0/multielo/blob/440f7922b90ff87009f8283d6491eb0f704e6624/multielo/multielo.py#L18

Hey @djcunningham0, loved the article. I'm implementing this for my friend's Catan games, though I'm trying to do it on Google Sheets -- I do not recommend it. Anyway, I figured I'd look to see how you handle ties and noticed this leftover comment from before v.0.3.0. You DO handle ties now! :)

djcunningham0 commented 2 years ago

Ha, good catch! I'll remove that comment next time I push some changes.

By the way, I'm not sure exactly what you mean by implementing it with Google Sheets, but I use Google Sheets to store my group's poker data and then I use this package to calculate Elo ratings. I.e., the results history is stored in the spreadsheet and then I have a web app that reads in the data and calculates the Elo history. The code for the app is in my poker-elo-dashboard repo (the the specific lines for reading the data from Google Sheets are here).

But if instead you mean you're doing all the Elo calculations with formulas in the spreadsheet... good luck to you!

fab0i commented 2 years ago

I did mean implementing the Elo calculations/formulas in Google Sheets! Honestly, it wasn't too bad, thanks to your article and this well-documented repo! The spreadsheet took about 5h in total, and I have almost no experience with spreadsheets (thankfully, I'm good enough at Googling things). If you want to see it, I created a copy here. It's still not finished, but I'm gonna try to do the actual leaderboard tab with graphs and whatnot today.