https://bugs.kde.org/show_bug.cgi?id=469991

Albert Astals Cid <aa...@kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|https://invent.kde.org/game |https://invent.kde.org/game
                   |s/kpat/-/commit/cc62c5599a7 |s/kpat/-/commit/31aa7a28cb6
                   |bd586ec94f874ab89efc50253d6 |3c9d7be35371a63ec2f481bec6e
                   |0c                          |fe

--- Comment #7 from Albert Astals Cid <aa...@kde.org> ---
Git commit 31aa7a28cb63c9d7be35371a63ec2f481bec6efe by Albert Astals Cid, on
behalf of Stephan Kulow.
Committed on 12/06/2023 at 21:38.
Pushed by aacid into branch 'release/23.04'.

Solve the simple simon autodrop problem less aggressively

Using the brute force algorithm for auto drop has several drawbacks
that users reported:
  - it's too aggressive (didn't matter for simple simon when I implemented
    it, as there is only one kind of foundation move)
  - the hint function is no longer a way to cheat your scores, you actually
    need to pick the winning move yourself

So revert that part and make in return the simple simon solver more useful
for autodrop by providing the right priority for foundation moves. In addition
I had to make sure we always return a first move even if the old interface
is not implemented.
(cherry picked from commit cc62c5599a7bd586ec94f874ab89efc50253d60c)

M  +0    -5    src/dealer.cpp
M  +12   -2    src/patsolve/abstract_fc_solve_solver.cpp
M  +1    -1    src/patsolve/abstract_fc_solve_solver.h
M  +4    -2    src/patsolve/simonsolver.cpp
M  +0    -15   src/patsolve/simonsolver.h

https://invent.kde.org/games/kpat/-/commit/31aa7a28cb63c9d7be35371a63ec2f481bec6efe

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to