Summary: | AI plays seriously bad moves against all rules and sanity, game has 33 cards and 130 points. | ||
---|---|---|---|
Product: | [Applications] lskat | Reporter: | Martin Walch <walch.martin> |
Component: | general | Assignee: | martin |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | iandw.au, kde-games-bugs |
Priority: | NOR | ||
Version: | 1.40 | ||
Target Milestone: | --- | ||
Platform: | Gentoo Packages | ||
OS: | Linux | ||
Latest Commit: | http://commits.kde.org/lskat/46f3b313bf91c9828c63e5be7a8f319254528647 | Version Fixed In: | 4.13.1 |
Sentry Crash Report: | |||
Attachments: |
screenshot of the game after first move
Comment out the condition on creating a new AI. Always create it. |
Description
Martin Walch
2014-01-23 01:23:29 UTC
Created attachment 84809 [details]
screenshot of the game after first move
I think I finally found out what probably has happened: when starting a new game while the previous game is not over and a move is currently in progress, the new game starts in an inconsistent state. So to have a chance of reproducing the bug do this: 1. start a new game against ai player 2. make a turn 3. while the animation of the first or the second part of the turn is running press Ctrl+N to start a new game Depending on when during a move you start the new game and how the cards are distributed, all sorts of strange effects may happen in the new game. (There is also a chance that nothing goes wrong.) Created attachment 86446 [details]
Comment out the condition on creating a new AI. Always create it.
(In reply to comment #3) > Created attachment 86446 [details] > Comment out the condition on creating a new AI. Always create it. I have tested this patch several times in several ways and sanity was always restored. (In reply to comment #4) > (In reply to comment #3) > > Created attachment 86446 [details] > > Comment out the condition on creating a new AI. Always create it. > > I have tested this patch several times in several ways and sanity was always > restored. Thank you for that fix. With your patch applied I can not reproduce the problem anymore. When reporting this bug I did not recognize bug #228067 as the same, but with the new insights how to reproduce the bug, I am pretty sure this is a duplicate of it. (So, the first to document this bug were Raj Mathur and Eckhart Wörner.) However, now that the fix is attached to this bug report I hesitate to close it as duplicate of the older report #228067. Git commit 922d9ced78f6eb319d1049795acef3d8e243fb0b by Ian Wadham. Committed on 07/05/2014 at 21:58. Pushed by ianw into branch 'master'. Related: bug 228067 LSkat AI would play illogically after you hit New during a move. M +1 -1 src/mainwindow.cpp http://commits.kde.org/lskat/922d9ced78f6eb319d1049795acef3d8e243fb0b Git commit 46f3b313bf91c9828c63e5be7a8f319254528647 by Ian Wadham. Committed on 07/05/2014 at 21:58. Pushed by ianw into branch 'KDE/4.13'. Related: bug 228067 LSkat AI would play illogically after you hit New during a move. M +1 -1 src/mainwindow.cpp http://commits.kde.org/lskat/46f3b313bf91c9828c63e5be7a8f319254528647 |