Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Atomic castling with touching kings #176

Closed
thomas-daniels opened this issue Sep 2, 2019 · 0 comments
Closed

Atomic castling with touching kings #176

thomas-daniels opened this issue Sep 2, 2019 · 0 comments

Comments

@thomas-daniels
Copy link
Member

This paper, written by FICS player Rekursiv, outlines all atomic rules as employed on FICS, and Lichess is using the same ruleset.

Quoting the paper:

As in normal chess, it is illegal to castle if your king has previously moved or if your rook with which you are castling has previously moved. It is also illegal to castle if you are currently in atomic check, if moving the king one square closer to the rook (with the rook staying put) would place you in atomic check (i.e. you cannot castle through check), or if completing castling would place you in atomic check.

Now consider this position: https://lichess.org/analysis/atomic/8/8/8/8/8/8/5k2/R3K2r_w_Q_-

Lichess allows O-O-O in here, which is inconsistent with this ruleset ("illegal [...] if moving the king one square closer to the rook (with the rook staying put) would place you in atomic check").

(Note that, when changing this, we must ensure that O-O-O stays valid in https://lichess.org/analysis/atomic/8/8/8/8/8/8/4k3/R3K2r_w_Q_- - Lichess correctly allows O-O-O in here.)

cc @niklasf

niklasf added a commit to niklasf/scalachess that referenced this issue Sep 3, 2019
niklasf added a commit to niklasf/scalachess that referenced this issue Sep 3, 2019
niklasf added a commit to niklasf/scalachess that referenced this issue Sep 3, 2019
niklasf added a commit to niklasf/scalachess that referenced this issue Sep 3, 2019
@ornicar ornicar closed this as completed in 9784172 Sep 3, 2019
ornicar added a commit that referenced this issue Sep 4, 2019
* master:
  v8.6.25
  add another atomic castling test position
  fix atomic castling with touching kings (closes #176)
  add failing test for castling through atomic check (#176)
  add regression test for atomic castling (#176)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant