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.

Sign up for GitHub

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

Jump to bottom

Preview of the "storm" ahead #164

Closed
CounterPly opened this issue May 2, 2018 · 1 comment
Closed

Preview of the "storm" ahead #164

CounterPly opened this issue May 2, 2018 · 1 comment
Labels
development info

Comments

@CounterPly
Copy link
Collaborator

@tthsqe12 @lantonov

This is just a heads up regarding the pawn-storm shenanigans in the coming patches:

The next patch...
59d10374 Blocked pawn storm

Gets completely reverted after these two patches...
29070bd0 Simplification: remove pawn shelter/storm masks
73e8daa1 Simplify the shelter mask

Also is closely related to these three patches...
c794c8c8 Simplify the ShelterStrength[] array
f7cc0026 Simplify BlockedByKing in pawn storms
12ef8f71 Use special rule for BlockedByKing

I am having a bit of trouble with 59d10374 for some reason. I'll post what I have done so far in a PR tomorrow if I still can't get the benches to match.
@CounterPly
Copy link
Collaborator Author

I'll post what I have done so far in a PR tomorrow if I still can't get the benches to match.

See #165.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
development info
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@CounterPly