Ticket #131 (closed defect: fixed)

Opened 6 years ago

Last modified 5 years ago

restricted_genmove answers elsewhere at fourlib-depth 9

Reported by: alain Owned by: gnugo
Priority: lowest Milestone: 3.7.11
Component: source Version:
Severity: trivial Keywords:
Cc: patch: yes

Description

With a fresh downloaded gnugo-3.7.9 from gnugo site, compiled with gcc-4.0.2 (fedora 4)
The test is gunnar:30 

# F18 should be a better and bigger way to live than B18.
loadsgf games/nngs/gnugo-3.3.18-overziel7-200304281000.sgf 110
30 restricted_genmove white F18 B18
#? [F18]

And the answer is elsewhere at fourlib-depth 9, but correct at 8 or 7 (default) 

./regress.pike -v -o "--fourlib-depth 9 --mode gtp" gunnar:30
gunnar:30       FAIL N19 [F18]

Valgrind says nothing more than (malloc/free: 76627 allocs, 76598 frees)

Attachments

restricted_genmove_7_10.1.diff Download (1.1 KB) - added by arend 5 years ago.
Trivial fix for restricted_genmove

Regression Results

Attachment Rev. PASS FAIL Nodes Status
restricted_genmove_7_10.1.diff Download never tested

Change History

comment:1 Changed 6 years ago by alain

  • Summary changed from restricted_genmove anwers elswhere at fourlib-depth 9 to restricted_genmove answers elswhere at fourlib-depth 9

comment:2 Changed 6 years ago by alain

  • Summary changed from restricted_genmove answers elswhere at fourlib-depth 9 to restricted_genmove answers elsewhere at fourlib-depth 9

comment:3 Changed 6 years ago by alain

At fourlib-depth 9:

  • gnugo-3.4 gives B18, one of the 2 possible answers. B18 is in the top_moves
  • gnugo-3.6 PASS, which is not allowed! No possible answer is in the top moves.
  • gnugo-3.7.9 N19 which is not allowed! No possible answer is in the top moves.

N19 is an endgame move when best move seems to be PASS. So it's ok to says the test fails.
Getting a PASS for answer would be more clear.

OT: it also means that fourlib-depth 9 does not work so well without some other depth changes.

comment:4 Changed 6 years ago by alain

  • Priority changed from normal to lowest
  • Severity changed from normal to trivial

Changed 5 years ago by arend

Trivial fix for restricted_genmove

comment:5 Changed 5 years ago by arend

  • patch set

comment:6 Changed 5 years ago by gunnar

  • Status changed from new to closed
  • Resolution set to fixed
  • Milestone changed from 3.8 to 3.7.11

The patch was applied long ago and I don't see any remaining reason to keep the ticket open.

Note: See TracTickets for help on using tickets.