mirror of
https://github.com/sockspls/badfish
synced 2025-04-30 16:53:09 +00:00
e88ef801af
Use ralpha instead of rbeta * rbeta is confusing people. It took THREE attempts to code razoring at PV nodes correctly in a recent test, because of the rbeta trick. Unnecessary tricks should be avoided. * The more correct and self-documenting way of doing this, is to say that we use a zero window around alpha-margin, not beta-margin. The fact that, because we only do it at PV nodes, alpha happens to be beta-1 and that the current stuff with rbeta works, may be correct, but is confusing. Remove the misleading and partially erroneous comment about returning v + margin: * comments should explain what the code does, not what it could have done. * this comment is partially wrong in saying that v+margin is "logical", and that it is "surprising" that is doesn't work. From a theoretical perspective, at least 3 ways of doing this are equally defendable: 1/ fail hard: return alpha: The most conservative. We bet that the search will fail low, but we don't know by how much and don't want to take risks. 2/ aggressive fail soft: return v (what the current code does). This corresponds to normal fail soft, with the added assumption that we don't care about the reduction effect (see below point 3/) 3/ conservative fail soft: return v + margin. If the reduced search (qsearch) gives us a score <= v, we bet that the non reduced search will give us a score <= v + margin. * Saying that 2/ is "logical" implies that 1/ and 3/ are not, which is arguably wrong. Besides, experimental results tell us that 2/ beats 3/, and that's not something we can argue against: experimental results are the only trusted metric. * Also, with the benefit of hindsight, I don't think the fact that 2/ is better than 3/ is surprising at all. The point is that it is YOUR turn to move, and you are assuming that by NOT playing (and letting the opponent capture your hanging pieces in QS) you cannot generally GAIN razor_margin(depth). No functional change. |
||
|---|---|---|
| .. | ||
| benchmark.cpp | ||
| bitbase.cpp | ||
| bitboard.cpp | ||
| bitboard.h | ||
| bitcount.h | ||
| book.cpp | ||
| book.h | ||
| endgame.cpp | ||
| endgame.h | ||
| evaluate.cpp | ||
| evaluate.h | ||
| main.cpp | ||
| Makefile | ||
| material.cpp | ||
| material.h | ||
| misc.cpp | ||
| misc.h | ||
| movegen.cpp | ||
| movegen.h | ||
| movepick.cpp | ||
| movepick.h | ||
| notation.cpp | ||
| notation.h | ||
| pawns.cpp | ||
| pawns.h | ||
| platform.h | ||
| position.cpp | ||
| position.h | ||
| psqtab.h | ||
| rkiss.h | ||
| search.cpp | ||
| search.h | ||
| thread.cpp | ||
| thread.h | ||
| timeman.cpp | ||
| timeman.h | ||
| tt.cpp | ||
| tt.h | ||
| types.h | ||
| uci.cpp | ||
| ucioption.cpp | ||
| ucioption.h | ||