1
0
Fork 0

Use MovePicker's move ordering also at root

After testing on our russian cluster: +3 elo after 4200 games

So keep it becuase it allows a good semplification.

Signed-off-by: Marco Costalba <mcostalba@gmail.com>
sf_2.3.1_base
Joona Kiiski 2010-08-19 13:29:40 +03:00 committed by Marco Costalba
parent 79a28841f9
commit a44f79141e
1 changed files with 26 additions and 2 deletions

View File

@ -117,7 +117,7 @@ namespace {
struct RootMove {
RootMove() { nodes = cumulativeNodes = ourBeta = theirBeta = 0ULL; }
RootMove() { mp_score = 0; nodes = cumulativeNodes = ourBeta = theirBeta = 0ULL; }
// RootMove::operator<() is the comparison function used when
// sorting the moves. A move m1 is considered to be better
@ -125,11 +125,12 @@ namespace {
// have equal score but m1 has the higher beta cut-off count.
bool operator<(const RootMove& m) const {
return score != m.score ? score < m.score : theirBeta <= m.theirBeta;
return score != m.score ? score < m.score : mp_score <= m.mp_score;
}
Move move;
Value score;
int mp_score;
int64_t nodes, cumulativeNodes, ourBeta, theirBeta;
Move pv[PLY_MAX_PLUS_2];
};
@ -143,6 +144,8 @@ namespace {
public:
RootMoveList(Position& pos, Move searchMoves[]);
void set_mp_scores(const Position &pos);
int move_count() const { return count; }
Move get_move(int moveNum) const { return moves[moveNum].move; }
Value get_move_score(int moveNum) const { return moves[moveNum].score; }
@ -739,6 +742,7 @@ namespace {
while (1)
{
// Sort the moves before to (re)search
rml.set_mp_scores(pos);
rml.sort();
// Step 10. Loop through all moves in the root move list
@ -2777,6 +2781,26 @@ namespace {
}
void RootMoveList::set_mp_scores(const Position &pos)
{
MovePicker mp = MovePicker(pos, MOVE_NONE, ONE_PLY, H);
Move move;
int moveCount = 0;
while ((move = mp.get_next_move()) != MOVE_NONE)
{
moveCount++;
for (int i = 0; i < count; i++)
{
if (moves[i].move == move)
{
moves[i].mp_score = 512 - moveCount;
break;
}
}
}
}
// RootMoveList simple methods definitions
void RootMoveList::set_move_nodes(int moveNum, int64_t nodes) {