search

ASPP / pelita

Actor-based Toolkit for Interactive Language Education in Python
https://github.com/ASPP/pelita_template
Other
63 stars 68 forks source link

BF: TK UI shows wrong homezone for blue team #804

Closed otizonaizit closed 3 months ago

otizonaizit commented 3 months ago

Fix longstanding bug introduced in 92ca5553f61e05c5edaf9f3bd240628bb1ef06f8

The homezone for the blue team shown in the field status (bottom right) is one square too big. Typical off-by-one error ;-)

(see screenshot) screenshot1

otizonaizit commented 3 months ago

by the way, the failed check is completely unrelated... any idea why sometimes these checks do get a timeout?

Debilski commented 3 months ago

That’s what happens if you don’t have a dedicated function for everything. Thanks.

Debilski commented 3 months ago

by the way, the failed check is completely unrelated... any idea why sometimes these checks do get a timeout?

I assume there is a race condition in the cleanup of a broken Pelita player. Maybe even similar to the one we had in the Pelita server. Looking into it.