1to9only wrote:From
Revision of SE ratings and resolution rules:
I think some of lksudoku's UR/UL and BUG fixes are included in this release. These from patterns game 333 are misrated:
- Code: Select all
12...3..44....5.67...48..2...5....3...2...4..67....9......69....615.....73....... # 0333 76 4.9/3.4/2.6
12...3..44....1.56...74..8...9....3...2...6..73....5......87....759.....36....... # 0333 78 4.8/3.0/2.6
12...3..44....5.36...71..8...9....2...2...6..38....9......48....456.....93....... # 0333 84 6.0/1.2/1.2
94...2..16....1.85...87..6...1....9...4...6..83....2......16....254.....41....... # 0333 87 5.3/1.2/1.2
12...3..45....2.67...54..8...9....3...1...8..43....6......78....159.....97....... # 0333 92 4.7/4.7/2.6
Thanks for the feedback.
lksudokus fixes and enhancements are summarized here:
https://github.com/SudokuMonster/SukakuExplainer/wiki/lksudoku-SE-fixes The bug fixes regarding Unique loop detection, BUG detection and Unique Rectangle/Loop Type 3 set detection have been integrated after testing. As they are bug fixes we didn't integrate them as an option but straight into the code, hence, the discrepancy in rating for some puzzles.
The enhancements will be included fully in the next release (and can be tested currently by cloning the repository and compiling it). I'll do a final round of testing on the extremely helpful file you posted (when I find time to download it) using the "batch solver" options which intial testing show no change in rating ER/EP/ED and a significant improvement in Rating speed. If the results remain consistent as I'm expecting then a new release that includes them will follow
All enhancement will be included as an option:
- Code: Select all
--revisedRating=N (1: New rating scheme)
--batch=N (Batch solving mode 1: and 2: are 2 modes for batch solving)
--techs=TECHSTRING TECHSTRING: a string of 0s and 1s that will enable/disable available solving techniques
lksudoku's Chains length detection bug fix shows promising results with detecting shorter chains but there have been some puzzles where longer chains and therefore higher than original ratings given. Because of that It wasn't integrated (awaiting for a fix for the fix to takes place
)
tarek