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

Automated tests for several functions, various bug fixes, TT changes #78

Merged
merged 25 commits into from
Apr 12, 2024

Conversation

Dannyj1
Copy link
Owner

@Dannyj1 Dannyj1 commented Apr 12, 2024

  • Automated tests for perft, SEE and evaluation run by CI on every pull request
  • Valgrind memory tests and clang sanitizers run by CI on every pull request
  • Fix PST tables not being applied correctly, causing issues and asymmetry in evaluation when mirroring positions
  • Fixed move generation allowing illegal castling moves
  • Fixed rook on 7th rank evaluation and other evaluation terms relying on the RANK_7 constant
  • Changed zobrist hashing to be less bug-prone
  • Reduced the size of TT entries, allowing storage of more entries

Elo | 17.44 +- 10.06 (95%)
SPRT | 60.0+0.60s Threads=1 Hash=64MB
LLR | 1.03 (-2.94, 2.94) [0.50, 2.50]
Games | N: 2990 W: 1048 L: 898 D: 1044
Penta | [123, 308, 524, 376, 164]

Dannyj1 added 25 commits April 9, 2024 12:57
…aluation of positions when mirroring the position

Bench: 9212883
# Conflicts:
#	.github/workflows/release.yml
…y other evaluation involving the use of the RANK_7 constant

Bench: 8967695
Bench: 8939888
Bench: 8939888
Bench: 8939888
Bench: 8939888
@Dannyj1 Dannyj1 merged commit 203fd2a into master Apr 12, 2024
2 checks passed
@Dannyj1 Dannyj1 deleted the tests branch April 12, 2024 12:02
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.

1 participant