Skip to content
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

Minor fix: Sudoku (redesign grid constraint) #4

Open
joshuaguerin opened this issue Aug 8, 2023 · 1 comment
Open

Minor fix: Sudoku (redesign grid constraint) #4

joshuaguerin opened this issue Aug 8, 2023 · 1 comment

Comments

@joshuaguerin
Copy link
Owner

Current grid constraint is ugly and not terribly elegant. I think a solution is to be had that involves div or mod that greatly reduces the complexity of the final 2 constraints.

@joshuaguerin
Copy link
Owner Author

joshuaguerin commented Aug 8, 2023

I was correct. Div appears to correctly identify the grid location of a variable (or will once corrected arithmetically), removing the need for a reference to explicit blocks. Updated in 39dcfc6.

I like where this is a lot better. I'm not convinced that I cannot make this read slightly better, so I'm going to leave the issue open for future consideration with fresh eyes. I think I've at least reached "not embarrassing" territory.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant