Skip to content

Add is_pseudolegal function#87

Merged
wdotmathree merged 9 commits into
mainfrom
is-legal
Aug 2, 2025
Merged

Add is_pseudolegal function#87
wdotmathree merged 9 commits into
mainfrom
is-legal

Conversation

@wdotmathree
Copy link
Copy Markdown
Collaborator

@wdotmathree wdotmathree commented Aug 2, 2025

  • Implemented is_pseudolegal function to check if the cached TT move is legal before using it for an initial search in the future.
  • Added a unit testing framework along to test future non search related changes for validity before deployment.
  • Slightly revamped Makefile to streamline testing and debugging

TODO:

  • Clean up Makefiles and make sure they are robust
  • Improve unit testing coverage
  • Switch to a different testing framework (I would like more direct feedback on test cases)

@wdotmathree wdotmathree marked this pull request as ready for review August 2, 2025 23:07
Copy link
Copy Markdown
Owner

@kevlu8 kevlu8 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@wdotmathree wdotmathree marked this pull request as draft August 2, 2025 23:13
@wdotmathree wdotmathree changed the title Use cached TT move for initial search Add is_pseudolegal function Aug 2, 2025
@wdotmathree wdotmathree marked this pull request as ready for review August 2, 2025 23:17
@wdotmathree wdotmathree merged commit ec9e3c6 into main Aug 2, 2025
1 of 2 checks passed
@wdotmathree wdotmathree deleted the is-legal branch August 2, 2025 23:17
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.

2 participants