Skip to content

Commit

Permalink
Zero redundant material imbalance terms
Browse files Browse the repository at this point in the history
First coefficient in the SameColor array does an
equivalent job when folded into the LinearCoefficients
array.

All of the diagonal terms in the OppositeColor array
are redundant due to cancellation.

No functional change.
  • Loading branch information
ceebo authored and mcostalba committed Nov 5, 2013
1 parent d34bb88 commit 1064288
Showing 1 changed file with 8 additions and 8 deletions.
16 changes: 8 additions & 8 deletions src/material.cpp
Expand Up @@ -39,11 +39,11 @@ namespace {
const Value RedundantRook = Value(554);

// pair pawn knight bishop rook queen
const int LinearCoefficients[6] = { 1817, -162, -1122, -190, 105, 26 };
const int LinearCoefficients[6] = { 1824, -162, -1122, -190, 105, 26 };

const int QuadraticCoefficientsSameColor[][PIECE_TYPE_NB] = {
// pair pawn knight bishop rook queen
{ 7 }, // Bishop pair
{ 0 }, // Bishop pair
{ 39, 2 }, // Pawn
{ 35, 271, -4 }, // Knight
{ 7, 105, 4, 7 }, // Bishop
Expand All @@ -54,12 +54,12 @@ namespace {
const int QuadraticCoefficientsOppositeColor[][PIECE_TYPE_NB] = {
// THEIR PIECES
// pair pawn knight bishop rook queen
{ 41 }, // Bishop pair
{ 37, 41 }, // Pawn
{ 10, 62, 41 }, // Knight OUR PIECES
{ 57, 64, 39, 41 }, // Bishop
{ 50, 40, 23, -22, 41 }, // Rook
{ 106, 101, 3, 151, 171, 41 } // Queen
{ 0 }, // Bishop pair
{ 37, 0 }, // Pawn
{ 10, 62, 0 }, // Knight OUR PIECES
{ 57, 64, 39, 0 }, // Bishop
{ 50, 40, 23, -22, 0 }, // Rook
{ 106, 101, 3, 151, 171, 0 } // Queen
};

// Endgame evaluation and scaling functions accessed direcly and not through
Expand Down

0 comments on commit 1064288

Please sign in to comment.