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

Update reverse move stats #2294

Closed
wants to merge 1 commit into from

Conversation

locutus2
Copy link
Member

For a good quiet non-pawn move consider the reverse move as bad and update the main history with a negative stat bonus.

STC:
LLR: 2.95 (-2.94,2.94) [0.50,4.50]
Total: 19292 W: 4401 L: 4141 D: 10750
http://tests.stockfishchess.org/tests/view/5d7751d50ebc594e7864973c

LTC:
LLR: 2.95 (-2.94,2.94) [0.00,3.50]
Total: 111952 W: 18762 L: 18275 D: 74915
http://tests.stockfishchess.org/tests/view/5d7771cf0ebc594e786498fa

Bench: 3914238

@locutus2
Copy link
Member Author

locutus2 commented Sep 11, 2019

I have running a second LTC with another version but currently it looks not like this will be pass or even better:

http://tests.stockfishchess.org/tests/view/5d78cc840ebc5902d384fcec

I include in this PR also the idea of @snicolet of introducing a reverse_move function which then used in two places.

Edit:
I stopped the above test because it was already below -2.4 after 75K games.

@snicolet snicolet closed this in 61f44ce Sep 11, 2019
@snicolet
Copy link
Member

Merged via 61f44ce, congrats!

@xoto10
Copy link
Contributor

xoto10 commented Sep 11, 2019

Nice one, congrats!

@mstembera
Copy link
Contributor

@locutus2 @snicolet Did either of you test the idea mentioned here
locutus2/Stockfish@8fec883...b08a88e
to do the same thing around line 1116? I would submit on your behalf but don't want to duplicate.

@mstembera
Copy link
Contributor

@locutus2 @snicolet Sorry I completely misread the suggestion.

snicolet pushed a commit that referenced this pull request Nov 8, 2021
This simplifies the penalty for reverse move introduced in
#2294 .

STC:
LLR: 2.94 (-2.94,2.94) <-2.25,0.25>
Total: 81696 W: 20627 L: 20540 D: 40529
Ptnml(0-2): 221, 9390, 21559, 9437, 241
https://tests.stockfishchess.org/tests/view/618810acd7a085ad008ef1cc

LTC:
LLR: 2.95 (-2.94,2.94) <-2.25,0.25>
Total: 44136 W: 11021 L: 10890 D: 22225
Ptnml(0-2): 28, 4570, 12746, 4691, 33
https://tests.stockfishchess.org/tests/view/61885686d7a085ad008ef20b

closes #3781

bench: 6547978
amchess added a commit to amchess/BrainLearn that referenced this pull request Nov 10, 2021
Tuned eval
Author: bmc4
Date: Mon Nov 8 13:14:18 2021 +0100
Timestamp: 1636373658

Simplify away the Reverse Move penalty

This simplifies the penalty for reverse move introduced in
official-stockfish/Stockfish#2294 .
amchess added a commit to amchess/ShashChess that referenced this pull request Nov 10, 2021
Improved learning and handicap mode (introduced handicap depth)
Tuned eval
Added polybook support
Stockfish patch
Author: bmc4
Date: Mon Nov 8 13:14:18 2021 +0100
Timestamp: 1636373658

Simplify away the Reverse Move penalty

This simplifies the penalty for reverse move introduced in
official-stockfish/Stockfish#2294 .
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

4 participants