Four more patches #177
Merged
Four more patches #177
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
5788691 is the official bench but 5788688 is correct since official-stockfish/Stockfish@59d1037 was skipped in asmFish. See discussion at lantonov#165 for more details.
5318575 is the official bench but 5318574 is correct since official-stockfish/Stockfish@59d1037 was skipped in asmFish. See discussion at lantonov#165 for more details.
5601228 is the official bench but 5601231 is correct since official-stockfish/Stockfish@59d1037 was skipped in asmFish. See discussion at lantonov#165 for more details.
5750110 is the official bench but 5750115 is correct since official-stockfish/Stockfish@59d1037 was skipped in asmFish. See discussion at lantonov#165 for more details.
CounterPly
added a commit
to CounterPly/asmFish
that referenced
this pull request
Jun 28, 2018
Four more patches
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.
Note:
Although these four benches look slightly different from those of SF, they are all correct. All benches were verified by recompiling Stockfish without this commit: official-stockfish/Stockfish@59d1037.
See discussion at #165 for more details on why this particular patch was skipped. Until now, the omission of "Blocked Pawn Storm" hasn't manifested in any bench differences (at the default depth of 13), but asmFish's bench should realign again with that of Stockfish after the commit official-stockfish/Stockfish@29070bd is reached.