Revert "Fix check for bishop pair in material imbalance"
authorMarco Costalba <mcostalba@gmail.com>
Thu, 5 Sep 2013 04:34:13 +0000 (06:34 +0200)
committerMarco Costalba <mcostalba@gmail.com>
Thu, 5 Sep 2013 04:34:48 +0000 (06:34 +0200)
commita30d3571ca07aab94675c3652f6bde15c339dfcf
tree4323183d07e0759f743379d9101dd54d7e4be7d4
parent457ac26de5e2925c87bdc0169478d138074c50f8
Revert "Fix check for bishop pair in material imbalance"

Idea is sound but implementation is partial. Ryan and Joona noticed that
    we leave an hole in material table. Also we got another report by an user
    of an odd behaviour. Namely, if you start stockfish and from the prompt
    give 'bench' you get 3453941, then if you run again bench you get 3453940.

    The reason is that two different positions with the same number of pieces,
    but one with a bishop pair and another without have the same material key.
    But after Eelco patch also different material imbalance and this yields
    to this issue.

    Restesting at long TC shows the patch does not really contribute at
    ELO improvement. Actually patch failed at long TC.

    LLR: -2.97 (-2.94,2.94)
    Total: 23109 W: 4104 L: 4092 D: 14913

    So revert.

    bench: 3453945
src/material.cpp