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

Simplify a candidate passer condition. #2607

Closed

Conversation

31m059
Copy link

@31m059 31m059 commented Mar 31, 2020

Prior to July 2019, we used a forward-file Bitboard to determine candidate passers. We simplified this with commit 13ba678, replacing the entire forward file with just a single blocking square.

However, a recent commit f2430bf by Michael Chaly (@Vizvezdenec) reintroduced a very similar condition, now applied to all passed pawns, not only candidate passers. Knowing the history of the candidate passed pawns' code, this makes one of the conditions mostly obsolete and redundant.

Its removal barely changes the bench number of nodes searched: out of more than 5 million, the value changes by less than 800. The removal also has no noticeable effect on the Elo performance of Stockfish.

Where do we go from here? In the past, tuning of the candidate passed pawns divisor showed some promise at STC. After this is merged, I will attempt a few tweaks of this constant.

STC:
LLR: 2.94 (-2.94,2.94) {-1.50,0.50}
Total: 31528 W: 6208 L: 6061 D: 19259
Ptnml(0-2): 541, 3673, 7205, 3788, 557
https://tests.stockfishchess.org/tests/view/5e825db0e42a5c3b3ca2ee21

LTC:
LLR: 2.94 (-2.94,2.94) {-1.50,0.50}
Total: 38546 W: 5083 L: 5009 D: 28454
Ptnml(0-2): 299, 3628, 11362, 3668, 316
https://tests.stockfishchess.org/tests/view/5e826ec7e42a5c3b3ca2ee2a

Bench: 5139561

@vondele vondele closed this in 375e4ee Apr 2, 2020
@vondele
Copy link
Member

vondele commented Apr 2, 2020

Thanks!

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.

None yet

2 participants