Shading/circle display error in 3/31/21 puzzle

[frozenpandaman]

NYT display:

Down for a Cross display:

[Plasser1]

This is happening in the the 4/18/21 NYT Puzzle as well https://downforacross.com/beta/play/13422 Correct circle placements: https://crosswordfiend.com/2021/04/17/sunday-april-18-2021/#ny

[stevenhao]

this must be a bug in the puz parser. i wonder if nyt changed something about their encoding.

[frozenpandaman]

@stevenhao found another likely .puz parser bug, in 4/21/2020:

Should be "R2D2" and "C3PO." Seems to think numbers should be Ts.

[cameronfreer]

See also #122

[JohnHKoh]

Does anyone have a .puz file they can share the demonstrates the incorrect behavior? That could help in getting to the bottom of this.

[ghost]

github doesn't allow me to attach .puz files in comments, but here's a file that demonstrates the circle issue when uploaded to DfA https://drive.google.com/file/d/1P_DehmxORtNi2F4rT8NbQCtE3xgIRfaw/view?usp=sharing

[stevenhao]

Thanks so much @JohnHKoh for the fix!

[stevenhao]

Turns out the bug was not in the parser, just in the frontend (did not support rectangular grids due to a bug that did a computation based on the number of rows instead of number of columns)