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

⚡ Refactor Update50movesRule() method #678

Merged
merged 5 commits into from Feb 24, 2024
Merged

Conversation

eduherminio
Copy link
Member

@eduherminio eduherminio commented Feb 23, 2024

  • Move Utils.Update50movesRule to Game, since it updates the 50 moves counter there
  • Increase bench depth to 8
  • Add two more fens to bench that involve triple repetition and 50 moves draw

8+0.08, [-5, 0]

Score of Lynx-refactor-update50movesrule-2685-win-x64 vs Lynx 2680 - main: 2485 - 2419 - 3100  [0.504] 8004
...      Lynx-refactor-update50movesrule-2685-win-x64 playing White: 1716 - 751 - 1536  [0.621] 4003
...      Lynx-refactor-update50movesrule-2685-win-x64 playing Black: 769 - 1668 - 1564  [0.388] 4001
...      White vs Black: 3384 - 1520 - 3100  [0.616] 8004
Elo difference: 2.9 +/- 6.0, LOS: 82.7 %, DrawRatio: 38.7 %
SPRT: llr 2.9 (100.4%), lbound -2.25, ubound 2.89 - H1 was accepted

@eduherminio eduherminio changed the title Refactor Update50movesRule() method ⚡ Refactor Update50movesRule() method Feb 23, 2024
@eduherminio eduherminio marked this pull request as ready for review February 24, 2024 16:27
@eduherminio eduherminio merged commit e1e2a7b into main Feb 24, 2024
28 checks passed
@eduherminio eduherminio deleted the refactor/update50movesrule branch February 24, 2024 16:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant