lynx-chess / Lynx

Chess Engine, written in C#
https://lichess.org/@/Lynx_BOT
MIT License
47 stars 4 forks source link

⚡ Remove `Move.CapturedPiece` #1020

Open eduherminio opened 1 month ago

eduherminio commented 1 month ago

It gets cumbersome for capture history, since we gotta save visited moves and captured piece. Benchs don't match, though. Something is still of though, since bench doesn't match.

Score of Lynx-perf-board-bitboard-remove-move-capturedpiece-3932-win-x64 vs Lynx-perf-board-bitboard-remove-makemovecalculatingcapturedpiece-3929-win-x64: 1568 - 1690 - 2540  [0.489] 5798
...      Lynx-perf-board-bitboard-remove-move-capturedpiece-3932-win-x64 playing White: 1197 - 405 - 1297  [0.637] 2899
...      Lynx-perf-board-bitboard-remove-move-capturedpiece-3932-win-x64 playing Black: 371 - 1285 - 1243  [0.342] 2899
...      White vs Black: 2482 - 776 - 2540  [0.647] 5798
Elo difference: -7.3 +/- 6.7, LOS: 1.6 %, DrawRatio: 43.8 %
SPRT: llr -2.26 (-78.2%), lbound -2.25, ubound 2.89 - H0 was accepted