fast_frandom() always returns float, not floating_t

Noticed by Michael Williams.

JSON reporting: Report finally picked move

The 'move' line now includes a 'choice' field with the coordinate (or
'pass' or 'resign') instead of a 'best' field which was rather
misleading in this case.

Prompted by Jonathan Chetwynd.

Revert "uct_genmove(): In case of json reporting, print a JSON final_decision message"

This reverts commit de9b9974efe249918544e971d2d5a8c6791525a4.

Jonathan convinced me that final_decision is a rather ugly hack,
and maybe we can report this in JSON differently without too much
invasiveness.

Merge branch 'master' of ssh://repo.or.cz/srv/git/pachi

uct_genmove(): In case of json reporting, print a JSON final_decision message

Add option -c chat_file to support fancy replies to kgs-chat

Merge branch 'master' into derm10

UCT setup_state(): Allow first move to be white, just emit a warning

Distributed engine: exclude random moves with very few playouts

This might fix http://files.gokgs.com/games/2012/3/31/pachi2-tuke.sgf
where pachi resigned while winning by 1.5.

board_effective_handicap: use first_move_value for h1 games

In http://files.gokgs.com/games/2012/3/30/pachi2-bathory.sgf pachi thought
it was leading up to the end even though first_move_value was 1.

uctp_generic_choose: fix determination of the second best move

In t-regress/games/2012-03-21-pachi2-Hujisawa-3.sgf pass was the best
move and nbest2 was never set.

uct_pass_is_safe: only require us to capture dead opponents

We can deadlock if we also require the opponent to capture our dead stones.
See example t-regress/games/2012-03-21-pachi2-Hujisawa-3.sgf
Several bugs are exposed by this example, this is the first of several fixes.

Fix compilation error from 16d9c54

MAX_PATTERN_DIST: 6 -> 7

This does not make a measureable difference against Fuego but
could be helpful against humans.

[FEAT_CAPTURE] payloads: Include countstones payloads only in feature_payloads() and if enabled

feature_payloads(): Take pattern_setup inst. of pattern_config as first parameter

pattern.c: Introduce CAPTURE_COUNTSTONES_MAX

tools/sgf2gtp.py: Extended description, minor commandline option rename.

tools/sgf2gtp.py: alternative implementation of SGF to GTP converter
which takes SGF variants into consideration. Only the main variant is
played.

Merge branch 'master' into jmhack

patternscan: Fix crash when only a single game (or small) is fed in

The pattern for the first move in first game is not added, causing an assertion failure.

Merge branch 'master' into jmhack

pattern.[ch]: Added CAPTURE feature counting number of stones.

tools/pattern_byplayer.sh: Fix indentation

Merge remote-tracking branch 'origin/jmhack'

tools/pattern_byplayer.sh: Minor bugfix, enclose var in "".

Merge branch 'ladders'

MAX_PATTERN_DIST: 14 -> 6

Performance should be similar and we dramatically reduce number of
collisions and somewhat lower memory usage. The next step is to
regenerate spatial dictionary and main pattern file, which will be
slightly laborous and I will first solve collisions in a more reasonable
way.

t-regress: Add 2012-03-15-pachi2-Leech.sgf

Ladders: Set new defaults that fix most testcases and do not slow down Pachi

t-regress: Add 2012-03-16-IMC-pachi2.sgf

Merge remote branch 'origin/master' into ladders

t-regress: Add 2011-06-09-jinen-pachi30s-2.sgf

Avoid rave bonus for all ko threats not just the first one.

Give 0 or negative rave bonus to ko threats before taking the ko.

tools/pattern_byplayer.sh: Fixed attribute parsing when multiple atributes on one line.

Merge remote branch 'origin/master' into ladders

Merge branch 'master' of ssh://repo.or.cz/srv/git/pachi

uct_search_result(): Verbose debug print in case pass is considered unsafe

Sometimes, Pachi mysteriously refuses to pass and it is not a reproducible bug; maybe this will help shed some light on it?

tools/pattern_getdrops.pl: Tool for learning patterns for unexpected moves

tools/kgslog2gtp.pl: Tool for converting kgsGtp log to extended GTP stream

tools/pattern_bayes_gen.sh: Allow pattern harvesting from stdin

Patternscan play: If aux. argument is 0, do not generate pattern match on this play

GTP play: Add support for a trailing engine-specific argument

Merge branch 'master' of ssh://repo.or.cz/srv/git/pachi

Spatial: Deal gracefully with databases of spatials with too large radius

This will allow easy experiments with finding the optimal MAX_PATTERN_DIST.

pattern_pdict_init() sphcachehit: Use calloc2(), we need initialized memory

Patterns: Use unsigned where appropriate

Spatial: Accurate analysis of hashing function performance

strcasestr(): Custom implementation for WIN32

Pointed out by linfk.

UCT Plugins: Act dummy on WIN32

Makefile: Special WIN section

Use appropriate socket includes on WIN32

Patch by linfk.

logline(): Use inet_ntoa() instead of inet_ntop() on WIN32

Pointed out by linfk.

time_sleep(): WIN32-specific implementation without nanosleep()

Patch by linfk.

util.h: Include WIN32 compatibility wrappers for sleep() and __sync_fetch_and...()

Pointed out by linfk.

Get rid of alloca() usage - non-portable

Pointed out by linfk.

Merge branch 'master' into ladders

is_middle_ladder(): Also test for initial escape by counter-capturing

middle_ladder_walk(): Take next escape move as a parameter

is_border_ladder(): Fix shortage of libs test

UCT prior prune_ladders: Do not prune during ko fights

UCT json: Include PVs for all cans instead of single seq record

UCT jsonbig: Include information about the board coloring

UCT avg_score: Track avg score of last move search explicitly in the tree

Merge branch 'json'

UCT prior prune_ladders: New setting for completely removing ladder moves from the tree

Merge remote branch 'origin/ladders'

group_atari_check(): Save 1pt ko if it protects an eye

Moggy: New side pattern for making eyes

t-regress: Add 2012-01-17-pachi2-Soromon-3.sgf

New middle ladder reader, using temporary boards

Moggy middle_ladder: Fix interaction of priors and test_pattern3_here()

is_middle_ladder(), wouldbe_ladder(): Pass group information

Moggy middle_ladder: Support for turning off middle ladder reading

We plan to switch to a completely different (much more expensive but
more accurate) ladder reading algorithm, which will not be really usable
in playouts. So how much will we hurt ourselves by removing this from
playouts?

t-regress by-ladders: Mark correctly detected ladders as intermittent

I.e. our ladder heuristic actually correctly detected the ladder, but it
did not keep us from considering the move anyway since the prior
discouragement is not enough.

middle_ladder_walk(): Add extra debug prints

middle_ladder_walk(): Allow alternative near-edge termination of ladders

Merge branch 'patterns'

README: Refer to pachi.or.cz/pat/ wrt. patterns

UCT pattern prior: -1000 -> -400 based on Jean-loup's tuning

-1000 is very bad on a cluster. With 10min S.D. or even 20min S.D.
single-threaded on a modern CPU, -1000 is best, however - we will need
to come up with some auto-default mechanism here.

Our falseeye branch fixes falseeye misevaluations

Merge branch 'master' into falseeye

Merge branch 'master' of ssh://repo.or.cz/srv/git/pachi

t-regress: Add 2011-07-28-pachi2-pkunzip-2.sgf

t-regress: Add 2011-08-09-pachi2-BlueSpark.sgf

t-regress: Add 2011-08-08-Dallas-pachi2-2.sgf

t-regress: Add 2011-08-09-somrak-pachi2-2.sgf

t-regress: Add 2011-06-18-dorabon-pachi2-4.sgf

t-regress: Add 2011-07-28-pachi2-Novicer.sgf

test-game.sh: Ignore GC lines not beginning with a number

t-regress: Add 2011-08-24-StoneGrid-pachi2.sgf

t-regress: Add 2011-09-04-pachi2-stv.sgf

t-regress: Add 2011-08-12-xiaosugi-pachi2.sgf

t-regress: Add 2011-06-13-pachi30s-Jep.sgf

t-regress: Add 2011-06-10-pachi30s-samba-2.sgf

t-regress 2011-06-05-Zen19-pachi2.sgf: Add extra test for falseeye

t-regress: Add 2011-06-09-botkiller2-pachi30s.sgf

t-regress: Add 2011-01-15-pachi2-rollingon.sgf

t-regress: Another new classification 'dead'