lichess-org / lichobile

lichess.org mobile application
https://lichess.org/mobile
GNU General Public License v3.0
2.05k stars 317 forks source link

Some bugs found in Android version #472

Closed JuicyRyan closed 8 years ago

JuicyRyan commented 8 years ago
  1. Puzzle solution sound (Bug) There is no piece moving sound when viewing the correct puzzle solution by clicking the forward arrow (double right arrow).
  2. Premove appeard in Analysis board (Bug) Sometimes in analysis board, one side has made a move while the computer still considers this side has not moved. So the move from other side seems a "Premove". For instance, when I was playing the analysis board, I made a move as white. But the computer still considered white has not moved. And if now I tried to move black pieces, it will appear as a premove which should not appear in the analysis mode. At the same time, the board is actually freezed, no side can move. White is already moved though the computer thinks not. Black may try to move but it is considered as a premove. This situation can happen for black. One solution is to go back one move by clicking the backward arrow, and wait a few seconds, then doing exactly the same move actually won't cause any problem this time. I don't know how to reproduce this bug since I really didn't find a pattern. But it happens in a high frequency.
  3. No Caro-Kann Defense: Steinitz Variation in Board Editor (Bug) There is no Caro-Kann Defense: Steinitz Variation (a predefined starting position) in Board Editor. If I choose this variation, it will directly jump to the previous variation which is Caro-Kann Defense: Panov-Botvinnik Attack.
  4. Anaysis doesn't start immediately when going into Analysis board (Bug) When I get into the analysis board from a game or a board editor, the analysis doesn't start immediately even which side moves first is already decided. I have to make a tentative move then the analysis will start.
  5. Trying to go back when in choosing variant interface from "continue from here" (Bug) If I try to start a offline game (continue from here) from analysis board or board editor, and then if I click back button instead of "Play" button in the middle when in "choosing the variant" interface, the screen will freeze instead of go back to the previous board editor or analysis board.
  6. Changing variant in Analysis board (Feature Request) Can we be allowed to change chess variant when in Anlaysis board? Currently, there is no way to do variant analysis directly from the anlaysis board. And if I want to do a variant analysis I have to start a new variant game then click the analysis button in the new game. But it is very inconvenient.
  7. End game positions in board editor (Feature Request) Can we have some end game positions in addition to the current starting positions in board eidtor? It would really help to improve.

Thanks!

veloce commented 8 years ago

Thank you very much for this detailed bug report!

Here's the list of bugs to track resolution:

sepiropht commented 8 years ago

I made a pr for the bug 3, but there is the same issue in the web app, i suppose they share the same json.

I think i can also add some endgames positions if it useful.

JuicyRyan commented 8 years ago

Thank you for your reply! Some end positions would be great! Also, I don't know whether you guys can reproduce all bugs mentioned above. So don't hesitate to tell me if a bug can't be reproduced. For me, those bugs are real. The second bug of offline analysis is really annoying.

veloce commented 8 years ago

@sepiropht about bug 3, the json was copied, so the same PR should be done in https://github.com/ornicar/lila repo.

If endgame positions are useful I'll certainly accept a PR, my only concern is that the positions list may be too long with both opening & endgame positions.

sepiropht commented 8 years ago

Yes thanks! You accept my first pr to open source.

I will see if can set up the web app and do the same pr.

May be we can do another select for endgame and specify the first 'opening' and the second 'endgame' ?

veloce commented 8 years ago

Maybe yes, I don't see what else to do. Keep the 'clear board' in both and the standard setup only in openings I guess

veloce commented 8 years ago

@JuicyRyan about bug #2 I can't reproduce a blocking issue hère.

I do find that black moves are premoves sometimes when you are moving quickly enough, but this is not an issue since the premove results as a

veloce commented 8 years ago

... Typing the comment from my phone is not easy... So the premove ends up in a move and i'm not able to get a situation where the board is freezed. If you do have steps to get to a freezing board situation please post them here. Otherwise i consider this is a non blocking minor issue

JuicyRyan commented 8 years ago

screenshot_20161106-083620 (Premove in Analysis Board)

I do have a way to produce bug 2 in one situation 90% of the time. So for the Anroid version, first close the existing lichess app and then start a new one. Second, go directly to Analysis Board and perform the following moves "quickly". 1. e4 e5, 2. Nf3 Nc6. The bug will very likely to occur when you move the knight for white or black. You can see the screen shot is this situation. And I can almost produce it every time!

However, I don't know the cause of this bug. Because I also encounter this bug very often in Puzzles. It seems if I move a piece on board quickly that the record doesn't record this move (so replaying an existing game won't cause this), then this bug tends to occur. It can happen for both white and black.

I really don't think this is a minor issue since it almost happens every time when I play with Analysis Board, whether to start a new game without a record or in puzzles. And it is really annoying. This bug didn't exist back in 4.42. (I didn't experience any update during 4.42 and 4.7.0 so I don't know whether such problem exists.) It may be related to bug 4 but I'm not sure.

And by "freeze" I don't mean you can't do anything. I only mean that no side is able to make next move when this occurs. But I can still solve it manually by going back one move and do the exact same move a little bit slowly.

I really hope this bug can be fixed. Thanks! Merci! @veloce

veloce commented 8 years ago

Well I simply disabled premoves in analysis board. Hope that will solve your issue.

veloce commented 8 years ago

All bug fixed, closing.