Feature request: AutoPlay needs to be made concurrent about unciv HOT 6 OPEN

Opening a can of worms..

v1: Show blocking popup with our rotationg arrow animation and a cancel button while it does its job
v2: Break it into small, fast (<250ms) steps, drive from WorldScreen.update - if there's autoplay work to do, launch coroutine that does a step and finishes with worldScreen.shouldUpdate = true, possibly some launchOnGLThread stuff to show where something happened

from unciv.

Let's talk about next turn, since it had the same problem

How does next turn work?

• clone game state
• change worldscreen to be "read only" state - players can explore the current world but cannot take action
• process next turn for clone in another thread
• when other thread finishes, replace the world screen

Sounds like you want something similar - only in your case you can replace the world screen on every turn, but keep it "read only"

from unciv.

That is to say - add a onTurnComplete callback parameter to next turn
NextTurn should not be aware of UI directly, it just clones the game state on every turn end and calls the function - which can be on another thread
The latency cost of cloning the game state every turn is the major cost here

from unciv.

The way we made it "read only" in the past was by changing the viewing civ... Could we set the viewing civ to none and set the selected civ to the original civ like in Spectator?
The spectator needs to view each civ as if they were them without being able to modify them. We might be able to reuse this to apply the locking mechanism.

from unciv.

No, we just set isPlayersTurn to false

from unciv.

Why did I not realize this? AutoPlay already does that in order to lock the screen.

from unciv.