skycloudd / eccat

An experimental UCI chess engine written in Rust
MIT License
1 stars 1 forks source link

Incorrect UCI Notation of Castling Moves in PV #12

Closed xu-shawn closed 2 weeks ago

xu-shawn commented 2 weeks ago

Running

position startpos moves g1f3 d7d5 e2e3 b8c6 d2d4 g8f6 c2c4 e7e6 f1e2 f8d6
go depth 4

Gives the following output:

info depth 1 seldepth 4 time 3 score cp 2 nodes 78 nps 19989 hashfull 0 pv e1g1 e8h8 c4d5 e6d5
info depth 2 seldepth 7 time 4 score cp 2 nodes 243 nps 57552 hashfull 0 pv e1g1 e8h8 c4d5 e6d5
info depth 3 seldepth 7 time 4 score cp 2 nodes 612 nps 136134 hashfull 0 pv e1g1 e8h8 c4d5 e6d5
info depth 4 seldepth 8 time 5 score cp 10 nodes 1280 nps 253473 hashfull 0 pv e1g1 e8h8 b1c3 d5c4 e2c4
bestmove e1g1

The second move in PV is short-castle by black, and it is denoted as e8h8, which is incorrect. The correct representation of castling is e8g8, as the king ends up on the g8 square.

skycloudd commented 2 weeks ago
position startpos moves g1f3 d7d5 e2e3 b8c6 d2d4 g8f6 c2c4 e7e6 f1e2 f8d6
go depth 4
info depth 1 seldepth 4 time 1 score cp 2 nodes 78 nps 43746 hashfull 0 pv e1g1 e8g8 c4d5 e6d5
info depth 2 seldepth 7 time 4 score cp 2 nodes 247 nps 59670 hashfull 0 pv e1g1 e8g8 c4d5 e6d5
info depth 3 seldepth 7 time 7 score cp 2 nodes 616 nps 79870 hashfull 0 pv e1g1 e8g8 c4d5 e6d5
info depth 4 seldepth 8 time 13 score cp 10 nodes 1284 nps 92232 hashfull 0 pv e1g1 e8g8 b1c3 d5c4 e2c4
bestmove e1g1