-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Wrong PGN export from study with multiple comments per node #15939
Comments
There are multiple chapters that are exported wrong in that study. |
I cannot reproduce with the simplified example https://lichess.org/study/a27kFcjo/MrbgVpLQ copy PGN
Export chapter PGN
|
I looked at this the other day and there is definitely something wrong with our parser. Basically, this should work |
Note that the PGN exported from LiChess Tools in the client was OK, imported OK and yielded no issues. Which means at least getting the data from the server to the move tree works fine, and indeed the study works just fine. It's the string output that is the issue. |
@Siderite could you send me the exported pgn by LiChess Tools? |
copied to the offending node and all variations following it. But note that this is my own rendering of the node data, so it might not help much. |
Lol! I just realized that it's not importing it. Geez! Steered you wrong. I remember that it was fine, though. Yes, the children are all wrong. node with ply 23 has children with ply 24 and 25. |
ah, thanks any way! We may need to ask the user for the original pgn. But I'll try to debug it first. |
Exact URL of where the bug happened
https://lichess.org/study/GJ5NRo9a/t7QR44fO
Steps to reproduce the bug
What did you expect to happen?
Exported PGN is correct
What happened instead?
PGN fails to export move
12...a5
, resulting in an invalid PGNOperating system
Windows 11
Browser and version (or alternate access method)
Brave
Additional information
Particular node that causes the issue can be found at path "/?WG)8aP8GVNG8P?.>NF(6`N'*_b-=UM(']H&4^V=ENU19"
The simplified PGN to the move is this:
1. e4 e5 2. Nf3 Nf6 3. Nxe5 d6 4. Nf3 Nxe4 5. d4 d5 6. Bd3 Bd6 7. O-O O-O 8. c4 c6 9. Re1 Bf5 10. Qb3 Qd7 11. c5 Bc7 12. g3 {Torch v2: 839058Mn} {[#]Testing out the Nakamura-Nepo game from the 2024 Candidates!} a5 13. Nbd2
The text was updated successfully, but these errors were encountered: