Skip to content

Conversation

@Rocky640
Copy link

@Rocky640 Rocky640 commented Jan 26, 2020

Instead of computing the initiative bonus on the material score + dynamic score
compute it on (material score/2) + dynamic score,

Passed STC
http://tests.stockfishchess.org/tests/view/5e2c4945ab2d69d58394fa8f
LLR: 2.94 (-2.94,2.94) {-1.00,3.00}
Total: 39387 W: 7594 L: 7386 D: 24407
Ptnml(0-2): 658, 4519, 9165, 4649, 697

Passed LTC
http://tests.stockfishchess.org/tests/view/5e2c85ccab2d69d58394faa7
LLR: 2.95 (-2.94,2.94) {0.00,2.00}
Total: 32588 W: 4206 L: 3986 D: 24396
Ptnml(0-2): 244, 2909, 9738, 3111, 253

bench: 4958282
bench rebased on master #2514: 4765486

Calculate complexity without the material score component.
bench 5037936
take 2: use half material values for initiative
benh 4958282
using a suggestion by stéphane
bench: 4958282
@xoto10
Copy link
Contributor

xoto10 commented Jan 26, 2020

Looks like you have the wrong bench now?

@xoto10
Copy link
Contributor

xoto10 commented Jan 26, 2020

EDIT: It will be indeed 4765486 once we merge the latest master (Tweak RestrictedPiece bonus)

Ahhh, you need to rebase on the latest master. Ok.

bench: 4765486
- 100 ;

// Give more importance to non-material score
score = (score * 2 - materialScore) / 2;
Copy link

@anshulongithub anshulongithub Jan 27, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Rocky640 is the line above correct, just wanted to confirm as in the PR description it says,

Instead of computing the initiative bonus on the material score + dynamic score
compute it on (material score/2) + dynamic score

Also before refactoring, we had dynamicScore added explicitly to the score returned from initiative before being added and assigned to the existing score like so:
score += dynScore + initiative(dynScore * 2 + score);
Is that still being done?
Just wanted to make sure that the refactored changes are equivalent to the original code that passed.

Also the bench in the last commit, 4765486 is different from what is in the PR description, do we need to update the correct bench there? Please confirm.

Thank you.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

a) Bench after refactoring (but before rebasing) was same as the one tested on fishtest.
b) Bench after rebasing had been added to pr description.
Everything is fine.

@vondele vondele closed this in 6d0eabd Jan 27, 2020
@vondele
Copy link
Member

vondele commented Jan 27, 2020

Thanks!

This was referenced Jan 27, 2020
vondele pushed a commit that referenced this pull request Jan 27, 2020
Adjust initiative score by psqt/2 instead of materialScore/2 which simplifies #2516

Passed STC
http://tests.stockfishchess.org/tests/view/5e2e667dab2d69d58394fc73
LLR: 2.94 (-2.94,2.94) {-1.50,0.50}
Total: 23198 W: 4506 L: 4353 D: 14339
Ptnml(0-2): 396, 2615, 5380, 2728, 418

Passed LTC
http://tests.stockfishchess.org/tests/view/5e2ed75cab2d69d58394fcbf
LLR: 2.94 (-2.94,2.94) {-1.50,0.50}
Total: 8519 W: 1179 L: 1062 D: 6278
Ptnml(0-2): 50, 775, 2472, 843, 74

closes #2522

Bench:  4684459
snicolet pushed a commit to snicolet/Stockfish that referenced this pull request Jan 29, 2020
Instead of computing the initiative bonus on the material score + dynamic score
compute it on (material score/2) + dynamic score,

Passed STC
http://tests.stockfishchess.org/tests/view/5e2c4945ab2d69d58394fa8f
LLR: 2.94 (-2.94,2.94) {-1.00,3.00}
Total: 39387 W: 7594 L: 7386 D: 24407
Ptnml(0-2): 658, 4519, 9165, 4649, 697

Passed LTC
http://tests.stockfishchess.org/tests/view/5e2c85ccab2d69d58394faa7
LLR: 2.95 (-2.94,2.94) {0.00,2.00}
Total: 32588 W: 4206 L: 3986 D: 24396
Ptnml(0-2): 244, 2909, 9738, 3111, 253

closes official-stockfish/Stockfish#2516

Bench: 4765486
snicolet pushed a commit to snicolet/Stockfish that referenced this pull request Jan 29, 2020
Instead of computing the initiative bonus on the material score + dynamic score
compute it on (material score/2) + dynamic score,

Passed STC
http://tests.stockfishchess.org/tests/view/5e2c4945ab2d69d58394fa8f
LLR: 2.94 (-2.94,2.94) {-1.00,3.00}
Total: 39387 W: 7594 L: 7386 D: 24407
Ptnml(0-2): 658, 4519, 9165, 4649, 697

Passed LTC
http://tests.stockfishchess.org/tests/view/5e2c85ccab2d69d58394faa7
LLR: 2.95 (-2.94,2.94) {0.00,2.00}
Total: 32588 W: 4206 L: 3986 D: 24396
Ptnml(0-2): 244, 2909, 9738, 3111, 253

closes official-stockfish/Stockfish#2516

Bench: 4765486
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.