Skip to content

Commit

Permalink
Add new shatranj book
Browse files Browse the repository at this point in the history
  • Loading branch information
ianfab committed Jan 18, 2018
1 parent 305f810 commit fa64a3d
Showing 1 changed file with 19,925 additions and 0 deletions.
Loading

5 comments on commit fa64a3d

@alwey
Copy link

@alwey alwey commented on fa64a3d Jan 19, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cutechess accepts normal FEN format rnbkqbnr/pppppppp/8/8/8/8/PPPPPPPP/RNBKQBNR w - - 0 1 (default)
or the shortened format rnbkqbnr/pppppppp/8/8/8/8/PPPPPPPP/RNBKQBNR w 0 1 because Shatranj has neither castling nor en passant captures.

The format used here rnbkqbnr/pppppppp/8/8/8/8/PPPPPPPP/RNBKQBNR w - - gets accepted almost accidently. This does not make a difference at the moment. The move count is ignored currently.

However, this might change, because the FEN handling of cutechess is inconsistent for black positions and odd numbers of plies.
position fen <FEN black to move> moves <move1> <move2> <move3>.

@ianfab
Copy link
Owner Author

@ianfab ianfab commented on fa64a3d Jan 19, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@alwey Leaving out the move counts also works for other variants (is that a feature or should I not rely on that it works?), so is the problem the ambiguity between using the short notation and leaving out the move counts?

@alwey
Copy link

@alwey alwey commented on fa64a3d Jan 19, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ianfab: Leaving out move counts is not a feature. It just works because of the current implementation. The move counts may get important for some variants or certain tests. For now it may work as is but this can change suddenly, so it might be better to provide the counts (on occasion).

@alwey
Copy link

@alwey alwey commented on fa64a3d Jan 19, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

E.g. endgame books would need counts.

@ianfab
Copy link
Owner Author

@ianfab ianfab commented on fa64a3d Jan 19, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, thanks.

Please sign in to comment.