search

Q726kbXuN / nytxw_puz

Turn NY Times crosswords into Across Lite files
The Unlicense
41 stars 10 forks source link

Add support for puzzle notes #11

Closed jkboyce closed 2 years ago

jkboyce commented 2 years ago

This change checks if the puzzle contains any solving notes, and adds them to the output's 'notes' field.

Solving notes are not uncommon in themed puzzles. Example: nyt.py Chrome https://www.nytimes.com/crosswords/game/daily/2008/09/11 Sep1108.puz

Note about notes: When the NYT was publishing in .puz format, they would often put notes into the puzzle title. For example here is their own published version of the above Sep 11, 2008 puzzle. (In the JSON version this actually contains two notes; in the published .puz file one ended up in the title and the other in the notes field.) I'm guessing they did this for compatibility in case software ignored the notes field. However the solvers I've tested (macOS Black Ink and downforacross.com) both handle notes properly so I'm disinclined to put them in the title.

Q726kbXuN commented 2 years ago

So I wanted to take a quick moment to mention a big thanks to @jkboyce and @edsantiago . This marks the end of the items I wanted to get to in my personal todo list, which you two have managed to intuit.

I'm going to call this v1, and opened a couple of bugs for bigger items I have left on my list as bugs for anyone to tackle, or for me to get to one day.

Of course, feel free to keep adding whatever features you come up with!