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

WeakUnopposed penalty for backwards on file A or H #3275

Closed
wants to merge 3 commits into from
Closed

WeakUnopposed penalty for backwards on file A or H #3275

wants to merge 3 commits into from

Conversation

FauziAkram
Copy link
Contributor

@FauziAkram FauziAkram commented Dec 25, 2020

New Pull Request, with the updated bench after the recent merge.

Do not give the WeakUnopposed penalty for backwards on file A or H

The original idea comes from Lolligerhans, and a series of tunings and tests done by Fauzi.

Passed STC:
LLR: 2.96 (-2.94,2.94) {-0.25,1.25}
Total: 140864 W: 28127 L: 27660 D: 85077
Ptnml(0-2): 2529, 16660, 31735, 16831, 2677
https://tests.stockfishchess.org/tests/view/5fe39dec3932f79192d39673

Passed LTC:
LLR: 2.95 (-2.94,2.94) {0.25,1.25}
Total: 67568 W: 8993 L: 8590 D: 49985
Ptnml(0-2): 523, 6176, 19983, 6579, 523
https://tests.stockfishchess.org/tests/view/5fe3dd1b3932f79192d39693
Bench: 4157164

@anshulongithub
Copy link

anshulongithub commented Dec 25, 2020

@FauziAkram the build is failing cuz the bench that was added in the PR message is not same as what is being computed during the AppVeyor build.
image
I think you'll also need to add the right bench in the commit message.

@Lolligerhans
Copy link
Contributor

Good job! Amazed that you made it work. :)

@FauziAkram
Copy link
Contributor Author

@Lolligerhans Big thank to you also to always keep new ideas coming

@FauziAkram FauziAkram changed the title Update pawns.cpp WeakUnopposed penalty for backwards on file A or H Dec 26, 2020
@vondele vondele added the to be merged Will be merged shortly label Dec 31, 2020
@vondele vondele closed this in d21e421 Dec 31, 2020
joergoster pushed a commit to joergoster/Stockfish-old that referenced this pull request Jan 6, 2021
Do not give the WeakUnopposed penalty for backwards on file A or H

The original idea comes from Lolligerhans, and a series of tunings and tests done by Fauzi.

Passed STC:
LLR: 2.96 (-2.94,2.94) {-0.25,1.25}
Total: 140864 W: 28127 L: 27660 D: 85077
Ptnml(0-2): 2529, 16660, 31735, 16831, 2677
https://tests.stockfishchess.org/tests/view/5fe39dec3932f79192d39673

Passed LTC:
LLR: 2.95 (-2.94,2.94) {0.25,1.25}
Total: 67568 W: 8993 L: 8590 D: 49985
Ptnml(0-2): 523, 6176, 19983, 6579, 523
https://tests.stockfishchess.org/tests/view/5fe3dd1b3932f79192d39693

closes official-stockfish/Stockfish#3275

Bench: 4109336
BM123499 pushed a commit to BM123499/Stockfish that referenced this pull request Feb 22, 2021
Do not give the WeakUnopposed penalty for backwards on file A or H

The original idea comes from Lolligerhans, and a series of tunings and tests done by Fauzi.

Passed STC:
LLR: 2.96 (-2.94,2.94) {-0.25,1.25}
Total: 140864 W: 28127 L: 27660 D: 85077
Ptnml(0-2): 2529, 16660, 31735, 16831, 2677
https://tests.stockfishchess.org/tests/view/5fe39dec3932f79192d39673

Passed LTC:
LLR: 2.95 (-2.94,2.94) {0.25,1.25}
Total: 67568 W: 8993 L: 8590 D: 49985
Ptnml(0-2): 523, 6176, 19983, 6579, 523
https://tests.stockfishchess.org/tests/view/5fe3dd1b3932f79192d39693

closes official-stockfish#3275

Bench: 4109336
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
to be merged Will be merged shortly
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants