From 67f91bc5eaed821f35f09c9490a80200e1ab6951 Mon Sep 17 00:00:00 2001 From: Marco Costalba Date: Sun, 20 Oct 2013 09:55:12 +0200 Subject: [PATCH] Revert "Further increase safe checks bonus" This reverts commit 4bc2374450e30101392 for two reasons. First regression testing shows almost equal score: Before the patch: ELO: 49.75 +-2.5 (95%) LOS: 100.0% Total: 27205 W: 7113 L: 3244 D: 16848 After the patch: ELO: 48.87 +-2.9 (95%) LOS: 100.0% Total: 20860 W: 5478 L: 2563 D: 12819 Second, and more sensible to me, this patch increases safe check bonuses to 4 times their original value (!) and considering: - Values were already well tuned - Values are highly critical - King safety is highly critical, very TC dependent and very difficult to test - Our testing coverage is partial (self-testing, blitz times) I think is better to be safe than sorry and so I revert the patch. bench: 8440524 --- src/evaluate.cpp | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/src/evaluate.cpp b/src/evaluate.cpp index 246d0e7f..d87c5127 100644 --- a/src/evaluate.cpp +++ b/src/evaluate.cpp @@ -200,12 +200,12 @@ namespace { const int KingAttackWeights[] = { 0, 0, 2, 2, 3, 5 }; // Bonuses for enemy's safe checks - const int QueenContactCheck = 24; - const int RookContactCheck = 16; - const int QueenCheck = 12; - const int RookCheck = 8; - const int BishopCheck = 2; - const int KnightCheck = 3; + const int QueenContactCheck = 12; + const int RookContactCheck = 8; + const int QueenCheck = 6; + const int RookCheck = 4; + const int BishopCheck = 1; + const int KnightCheck = 2; // KingExposed[Square] contains penalties based on the position of the // defending king, indexed by king's square (from white's point of view). -- 2.39.2