-
Notifications
You must be signed in to change notification settings - Fork 63
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
Use different search expansion for top players #847
Conversation
Codecov Report
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems fine
self._failed_matching_attempts * config.LADDER_TOP_PLAYER_SEARCH_EXPANSION_STEP, | ||
config.LADDER_TOP_PLAYER_SEARCH_EXPANSION_MAX | ||
) | ||
else: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Else isn't strictly necessary here, but I don't really care in this situation. Maybe you could say it helps readability in this case
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll let it in for the people like me that find code hard to read
e8699fd
to
3ce4a6f
Compare
Closes #806
This should enable 2k+ players to match all other 2k players if they are closer than 400 rating. I asked Tagada again and we wanted to have it 2k+ instead of 1.8k+.