-
Notifications
You must be signed in to change notification settings - Fork 22
Description
Is your feature request related to a problem? Please describe.
I'm reviving an idea that some people were interested in, but development apparently stopped on (see this pull request). It still randomly chooses the next game, but gives each game weighted odds for being chosen so that games that haven't been seen in a while are more likely to appear.
Describe the solution you'd like
I'm not sure it's ready for a pull request (I'm not sure the proper Git Ettiquette for doing so), but I have a branch that adds this as an option. The code isn't "formatted" the same way as the rest (there's a solid page of comments for example), which would definitely need to be cleaned up before merging. The code comments and the messages that get printed to the log should help other people look over the code and make sure it's doing what it's supposed to and won't cause issues elsewhere.
Describe alternatives you've considered
There's not really an alternative, the method for choosing the next game would need to be modified.
Additional context
I'm sure something else will come up, but there's one change that I made that I don't REALLY understand. Initially, the settings screen wasn't "remembering" that I had picked Weighted Odds between sessions. I fixed this by changing setupform.lua
around line 338. I'm not sure if that's the -best- solution -- why is and 0 or -1 or -2
needed at all there?