Lila: ceval drop suggestion UI glitch

Created on 20 Dec 2016  路  14Comments  路  Source: ornicar/lila

On black's turn, a computer suggested drop is shown as a circled semitransparent black piece. When black moves, the drop suggestion momentarily switches colors to a white piece before disappearing.

bug can't reproduce no scala

All 14 comments

@niklasf

So far I am not able to reproduce this. Maybe I should try a specific position?

I can reproduce 100% of time, and it doesn't actually matter which side is moving. If the engine suggests a drop and a move is made, the drop suggestion briefly flashes the other color before disappearing. Doesn't matter whether the move is the suggestion or some other move.

hmm, maybe was recently fixed? I last checked on my home computer a couple days ago, will follow up shortly when I get home.

please reopen, problem still exists.

Video of issue, OS X chrome 56.

Niklas helped me investigate this today. The issue is in autoshape.js::compute, which:

  • gets suggestion color from chessground instead of deriving from ceval ply
  • doesn't verify ceval ply matches board ply before displaying a suggestion

This issue affects promotion suggestions and drops. Regular moves do not include a color in the suggestion, so do not flip. Nevertheless, the UI might become more responsive, even for regular move suggestions, if autoshape hid the suggestion as soon as a move was made (and ply didn't match ceval), instead of waiting for a ceval callback. Here's a relevant log snippet, from analyse/ctrl.js::setAutoshape, showing the color flip

computeAutoShapes [{"orig":"b5","brush":"paleBlue"},{"orig":"b5","piece":{"color":"white","role":"queen","scale":0.8}}]
computeAutoShapes [{"orig":"b5","brush":"paleBlue"},{"orig":"b5","piece":{"color":"black","role":"queen","scale":0.8}}]

Related: #2307, #2531

Niklas fixed it.

These were two different issues. I am on it though

Tough luck. There ~is~ might be one more case to be fixed (which is when the PV is not hovered).

Edit: Maybe ... I didn't manage to cause the glitch without hovering the PV. @isaacl: Can you please give it another try on latest master or once the fix is deployed?

Even with logging the auto shapes (let alone visually) I couldn't see anything wrong. @isaacl If it's different for you, please let me know.

I saw this yesterday, and was able to reproduce about 30% of the time when I clicked on the ceval drop suggestion. I guess it's not fully fixed?

It's worth checking if this is still an issue after the ui/analyse rewrite.

OK I'll reopen if I see it again

Was this page helpful?
0 / 5 - 0 ratings

Related issues

ShRyDeR picture ShRyDeR  路  3Comments

Cristian-A picture Cristian-A  路  3Comments

serversideapps picture serversideapps  路  3Comments

nojoking picture nojoking  路  4Comments

thomas-daniels picture thomas-daniels  路  4Comments