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

Sudoku has implementation issues. #56

Open
Anthony-Handwerker opened this issue Oct 15, 2014 · 2 comments
Open

Sudoku has implementation issues. #56

Anthony-Handwerker opened this issue Oct 15, 2014 · 2 comments
Assignees
Labels

Comments

@Anthony-Handwerker
Copy link
Collaborator

The two I've noticed are:
A) Advanced Deduction throws an exception upon being clicked.
B) The case rule always claims to be incorrectly applied.

@Bram28
Copy link
Owner

Bram28 commented Oct 16, 2014

There is a lot not right about the current Sudoku rules:

"Forced by Deduction" -> terrible name, since all rules are deductive. Instead, this should be renamed to something like "Only Number" or "Last Number" (for a specific cell)

"Forced by Elimination" -> also a bad name: many rules are based on elimination. This should be renamed to something like "Only spot" or "Last Spot" (for a specific number in some specific row/column/black)

"Advanced Deduction" -> I have no idea what this rule even does. And if it is too advanced, it should probably just be deleted.

There is a presentation for Sudoku in the misc/Rules folder: someone needs to develop those.

@saltzl
Copy link
Collaborator

saltzl commented Nov 1, 2014

I've started re-writing the rules to match the presentation. Also, I agree, we should get rid of Advanced Deduction.

@saltzl saltzl self-assigned this Nov 12, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants