Merge pull request #50 from lemonsqueeze/can_countercap

Can countercap

can_countercapture(): foolproof api

can_countercapture(): check snapback / can_countercapture_any(): no checks

t-unit: can_countercap

selfatari_cousin(): fixed invalid can_countercapture() calls

Merge pull request #49 from lemonsqueeze/dragon

Dragon

t-unit: two_eyes test suite

dragon.c: deal with virtually connected groups

Merge pull request #48 from lemonsqueeze/tactics_undo

Tactics undo

ladder: use undo

selfatari: use undo

1lib.c: don't use board clen in tactics

Merge pull request #47 from lemonsqueeze/board_undo

Board undo

board.c: merged board_undo.c

board_undo: define QUICK_BOARD_CODE to get compiler error with forbidden fields.

board_undo: define BOARD_UNDO_CHECKS to guard against invalid quick_play() / quick_undo() uses (disabled by default)

t-unit: board_undo stress test

board_undo.c: fast play() + undo() pair to try out a move.

can use with_move() statement to call underlying board_quick_play() / board_quick_undo() transparently.
Only core board structures are maintained, see board.h

Merge pull request #46 from lemonsqueeze/moggy_test

Moggy test

tools: scripts to convert board diagrams from board_print() to t-unit / gtp / sgf

t-unit: moggy testing

replay: replay_sample_moves(),
engine.h: engine_done()

uct: moved printhook_ownermap() logic to ownermap.c,
engine.h: engine_printhook -> engine_board_print, renamed typedefs to avoid clashes

Merge pull request #45 from lemonsqueeze/dead_stones_fix

Dead stones fix

Merge pull request #41 from avysk/msys-fixes

Msys fixes

dead stones: debug info

dead stones: add own unclear dead groups if it doesn't change the outcome.

dead stones: don't skip playout after 2 passes, breaks dead stones detection

dead stones: uct fix

rename DOUBLE_TIME to DOUBLE_FLOATING

Merge pull request #38 from lemonsqueeze/predict

Prediction rates

Merge pull request #42 from jdanekrh/patch-1

Travis CI is failing

Travis CI is failing

I noticed that the CI is failing for PR #41. It is because the installation of clang Ubuntu package is fetching a file from llvm.org and that file is no longer there, see http://stackoverflow.com/questions/37558151/travis-ci-build-failing for a discussion.

Immediate solution would be to temporarily remove clang installation from the build. Longer term solution depends what comes out of the debate referenced at that SO.com question.

Add MSYS2_64 switch in Makefile

To compile using msys2 / mingw64

Change DOUBLE to DOUBLE_TIME

DOUBLE conflicts with some system headers on Windows, causing build
errors if defined.

t-predict: test components prediction rates

gtp: added pachi-predict command

replay engine: sample moves from moggy

new params:

  runs=n      set number of playout runs to sample.
              use runs=1 for raw playout policy

  no_suicide  make sure engine doesn't allow group suicides
              (off by default)

dcnn engine: plays dcnn best move

replay, montecarlo: engine_done

Merge pull request #37 from lemonsqueeze/side_ladder_fix

t-unit + side ladder fixes

fixed side ladder bug (check countercaptures)

. . O . . . .
. . O X X X X
. O X O . . .
. O X X O . .
. X X O . . .
. O O . . . .
. . . . . . .

A3 is no ladder

t-unit: ladder test suite

t-unit: display stats at the end

t-unit: added sanity checks

t-unit: new board format (spaces !)

t-unit/to_new_format script can be used to convert old sar.t files

Merge pull request #36 from lemonsqueeze/dcnn54

Detlef's 54% dcnn

use Detlef's 54% dcnn

fixed README, this is Detlef's 44% dcnn actually

Merge pull request #34 from sthalik/pr/add-noreturn-to-thread-exit

add attribute noreturn to thread callbacks

Merge pull request #33 from sthalik/pr/fix-win32-no-regex

the C language doesn't allow for argument name omission

add attribute noreturn to thread callbacks

GNU CC complains about the return value without an actual return.

the C language doesn't allow for argument name omission

This fixes the case of missing regex.h on mingw-w64.

Merge pull request #31 from xerus/master

pattern3: Group commands in block after if clause

pattern3: Group commands in block after if clause

Avoid an extra calling pattern_gen()

Silence -Wmisleading-indentation

Merge pull request #28 from lemonsqueeze/dcnn

Use dcnn for root priors

Merge branch 'pull-req/win32' of https://github.com/sthalik/pachi

Conflicts:
tactics/nakade.c

Merge pull request #25 from sthalik/pull-req/tree-node-size-padding

reduce cache contention by removing struct tree_node padding

Merge pull request #26 from lemonsqueeze/tree_suicide_fix

tree_expand_node(): check group suicides

updated README

use dcnn for root priors

caffe dcnn support

Makefile: c++ rules

make g++ happy

Merge pull request #23 from sometimesfood/fix-compiler-warnings

Fix compiler warnings

silence warning for infinite loop with a return value

work around GCC reachability warning

GCC 5.3.0 doesn't seem to know that all call paths inside the switch
statement end in a return. Add the same assert(0) as in the default
case.

work around setsockopt prototype differences

win32 defines optval as (const char*) but any other system
as (const void*). It can work for both if val is a char.

fix GCC cast-to-narrower-type complaints

This came up with win64 systems.

util.h: add missing stdio usage for printf(3) family

chat.c: fix unsigned long/int mismatch

allow for build without libregex

This isn't included by default on mingw-w64 installations, regarded as
user contrib.

cf.
https://sourceforge.net/projects/mingw/files/Other/UserContributed/regex

fix win32 warning about subtraction by InterlockedExchangeAdd

It complains about overflow. It probably means that the result can be
unsigned.

always include winsock2.h before windows.h

Some files like distributed/*.c include winsock. It would be annoying to
change each one of them to

before #include "util.h"

So include winsock header all the time.

change mingw-w64 win32 unrecognized printf modifiers

GCC 5.3.0 win64 doesn't recognize them, despite them working in 4.9.3
win32 with no warnings.

tree_expand_node(): check group suicides

board_is_valid_play() allows things like:
  X X X .
  X O * X
use board_is_valid_play_no_suicide() instead

tree: serialize from correct offset wrt packing

stats.h: move floating_t up front

It can be typedef'd to a double causing redundant padding.

uct/tree.h: reduce padding for cache efficiency for struct tree_node

By moving larger members to the front, we reduce the struct size
from 96 to 88 bytes.

Before further improving cache efficiency, this itself nets a 11% gain
in the amount of evaluations.

% for p in pachi pachi-ok ; do
    for i in $(seq 1 10); do
      echo genmove b |
      nice -n -6 ./$p -t _500 threads=8 2>&1 |
      egrep -o '[0-9]+ games/s,'; echo $p $i done >&2;
    done |
    awk "{ sum += \$1 }; END { print \"$p \" sum }"
done

This is on an Intel i7 with 8 MB of L3 cache and HT enabled.

Use fabs instead of abs for floating point values

Fix GCC compiler warnings

Merge pull request #17 from jirkadanek/travis

Feature for skipping tests

joseki_check(): Fix fatal bug, pointed out by y-ich in issue #18

autotest: Fix gogui handicap patch link

t-unit/test.c: lines starting with exclamation marks are disabled tests

Previous commit bc21229 already Fixed #3

Merge pull request #16 from jirkadanek/travis

Fix CI and tests

Merge pull request #12 from lemonsqueeze/is_bad_selfatari

is_bad_selfatari() fixes !

Install newer clang in .travis.yml and do not allow failure

Skip a failing test in t-unit/sar.t

See Issue #3 for details

Merge pull request #14 from lemonsqueeze/gogui_gfx

GoGui gfx analyze commands

Merge pull request #15 from jirkadanek/travis

Configure Continuous Integration (Travis)

Merge pull request #13 from lemonsqueeze/autotest_show_diagram

autotest-show-diagram

unittest(): exit(EXIT_FAILURE) if some tests fail

Add .travis.yml configuration

gogui-analyze_commands (uct only)

gogui-live_gfx best_moves|best_seq|winrates

gogui-winrates

gogui-best_moves

gogui-owner_map