search

sigh / Interactive-Sudoku-Solver

Fast web-based Sudoku solver. Supports many variants including all grid sizes up to 16x16.
MIT License
33 stars 9 forks source link

add renban constraint. #1

Closed vfig closed 2 years ago

sigh commented 2 years ago

Thanks!

The way you handled renbans is clever.

vfig commented 2 years ago

renbans: less clever, more necessity. i tried to write an 'AllContiguous' constraint along the lines of 'AllUnique', but i couldnt understand how that layer of the thing works! so i fell back to the individual cell constraints.

i see that github picked up the whisper change into this pr too. hadnt intended that yet, but i guess its fine!

sigh commented 2 years ago

Ah, I see. I converted it into a dedicated handler in https://github.com/sigh/Interactive-Sudoku-Solver/commit/eb1afd30132754d227e1d7f0eb5157cc8f73fb36

Yeah, the design of the solver is not very well documented. I'd be happy to explain/document any specific details if you end up trying to do something similar in the future.

I also make some small tweaks to both your renban and whisper UI (in different commits).

vfig commented 2 years ago

yeah, i pulled those already. looking at your reban constraint handler changes has helped a bit. i will be adding some more rule types in a ghetto fashion just for my use, but if i think any of them are worth PRing, i will probably ask you for that explanation!