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.

Sign up for GitHub

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

Jump to bottom

角歩の不成を含む詰みが読めない #257

Closed
barcodehair opened this issue Dec 14, 2022 · 2 comments
Closed

角歩の不成を含む詰みが読めない #257

barcodehair opened this issue Dec 14, 2022 · 2 comments

Comments

@barcodehair
Copy link

角や歩の不成がからむと、短手数でも詰みが読めないことがあります。
GenerateAllLegalMoves はもちろんtrueにしています。

詰みが読めない局面の例
5B1n1/8k/6Rpp/9/9/9/1+p7/9/K8 b rb4g4s3n4l15p 1
7nk/9/7Gp/6+RP1/9/9/1+p7/9/K8 b Pr2b3g4s3n4l14p 1

バージョン7.0では問題ありませんでしたが、
以下のコミットから発生するようになったようです。

  • CAPTURES_PRO_PLUSの指し手生成バグってる問題。
    ca89b8b
@yaneurao
Copy link
Owner

ご報告、ありがとうございます。原因まで特定していただいて、感謝です!!😁
次回の改修の時に修正しますね。

yaneurao added a commit that referenced this issue Jun 21, 2023
@yaneurao
- 角歩の不成を含む詰みが読めない #257 問題を修正
b40959f 
 - #257
 - 以前のcommitで壊していた。rollbackした。
   - GenerateAllLegalMovesオプションをオンにした時に「駒を取る手 + 歩の成り」と「駒を取らない手 + 歩の不成」を分けて生成したいのだが、これが現状の指し手生成ルーチンでは難しい。templateの特殊化を頑張ればできるのだが、ここをそんなに頑張っても仕方ないので(ふだん使う機能ではないし、バグらないように書くのむずいし、どうせ指し手オーダリングがきちんとしていればほとんど効率落ちないし)いまのままにしておく。
@yaneurao
Copy link
Owner

早くからご報告いただいていたのに修正が遅くなって申し訳ありません。この壊したcommitの意図通りに修正するの難しそうだなーと思って放置してました。結局、難しいので元のコードにrollbackすることにしました。ご報告、ありがとうございました。

b40959f

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@barcodehair @yaneurao