-
-
Notifications
You must be signed in to change notification settings - Fork 203
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
Support impossible checks when validate position #446
Support impossible checks when validate position #446
Conversation
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.
Thanks for the PR, It looks correct, I have few comments about where it should be and some styles changes. Let me know if you struggle with any of these comments.
Move implementation to standard. Fix unsafe calls to get, head and tail.
I fixed the comments, moved the implementation to the standard variant, this made the file changes more but hopefully it's clearer. Also, checking for optional in multiple places made the look more complex than it actually is, please let me know if you have any more suggestion to improve them. |
868fcc8
to
f1710ab
Compare
Thanks @Oziomajnr, I took the liberty to do some refactor to make the logic more clear. Please check if you think it's good. I think we can improve it's a bit further, but I think it's good for now. |
Thanks, the changes looks good, thanks. |
Hey @Oziomajnr, FYI, this is a breaking change, so we're discussion the best way to integrated it with lila. So this is on hold atm. |
…sition * master: Specialize ByRole for bitboard.Board Specialize ByColor for bitboard.Board
Limited the fix to standard variant for now, as I need to test for other variants too.
Fixes #403