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

Fails to parse with 3 queens. Move "Qe1f2" #21

Open

Rairden opened this issue Dec 16, 2021 · 1 comment

Rairden commented Dec 16, 2021 •

edited

Loading

Fails to parse with 3 queens. The problem string is "Qe1f2".

Here is the move: https://lichess.org/pz8W3CdJ#118

To debug:

set breakpoint here: https://github.com/freeeve/pgn/blob/v1/pgn.go#L187
for condition: black == "Qe1f2"

The problem seems to be at the bottom of your findAttackingQueen method. Returns error ErrAmbiguousMove (pgn: ambiguous algebraic move).

Captures of length 5 seem to work fine:

Q3xh2
Qexg3
Qde3+

The text was updated successfully, but these errors were encountered:

Owner

freeeve commented Dec 16, 2021

Thanks for the report!

Rairden changed the title ~~Fails to parse with 2-3 queens. Move "Qe1f2"~~ Fails to parse with 3 queens. Move "Qe1f2"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment