New master #636
Merged
New master #636
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Author: Ben Koshy No functional change
No functional change.
It seems to be a waste of time to loop through all remaining root moves after finishing each PV line. This patch skips this until we have reached the last PV line (this is the way it was done in Glaurung and very early versions of Stockfish). No functional change in Single PV mode. MultiPV=3 STC and LTC tests LLR: 2.95 (-2.94,2.94) [0.00,5.00] Total: 3113 W: 1248 L: 1064 D: 801 LLR: 2.95 (-2.94,2.94) [0.00,5.00] Total: 2260 W: 848 L: 679 D: 733 Bench: 5023629
The difference between QCAPTURES_1 and QCAPTURES_2 quiescence search stages boils down to a simple check of depth. The way it's being done now is unnecessarily complex. This patch is simpler, clearer, and easier to understand. Passed SPRT[-3..1] test at STC: LLR: 2.95 (-2.94,2.94) [-3.00,1.00] Total: 99755 W: 22158 L: 22192 D: 55405 http://tests.stockfishchess.org/tests/view/5a71f41c0ebc590f2c86e9cb No functional change.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.