lichess-org / lila

♞ lichess.org: the forever free, adless and open source chess server ♞
https://lichess.org
GNU Affero General Public License v3.0
14.86k stars 2.21k forks source link

make lila.study.StudyPgnImport accept some illegal moves #15607

Open ornicar opened 2 weeks ago

ornicar commented 2 weeks ago

Namely, moves that fail to escape a check. It happens in tournaments and the game keeps going, but at the moment it stops our broadcast transmission. Example: https://lichess.org/broadcast/maia-chess-open-2023/round-5/nSX1fm5q/B5Fu3WBq Re3 was played and the game continued: https://view.livechesscloud.com/#894f1c51-4c15-458b-98f4-7b7dfa038c34/5/5

Siderite commented 2 weeks ago

But this could apply to any number of situations, like someone moving a queen as a knight or something. I think a checkbox should be used for illegal imports. And it will cause havoc with analysis.