search

ASPP / pelita_template

Default template to get started with the Pelita game.
3 stars 25 forks source link

adapt tests to the new layout strings from PR 706 #38

Closed otizonaizit closed 4 years ago

otizonaizit commented 4 years ago

This adapts the tests to the new layout strings from https://github.com/ASPP/pelita/pull/706

Debilski commented 4 years ago

No offence to your skills with the colour bucket tool but maybe we should add the noise-showing cells simply to the grid mode in Tk when we add the noisy ghost shadows? 🙃 Also, I think image12.png is not needed.

image
otizonaizit commented 4 years ago

That's the best I could do on my laptop screen in a reasonable time. It gets the message through. When and if we have a better visualization, you are welcome to push it. Until now, this is the best we have. (doing vs talking)

On 18 June 2020 13:54:10 CEST, Rike-Benjamin Schuppner notifications@github.com wrote:

No offence to your skills with the colour bucket tool but maybe we should add the noise-showing cells simply to the grid mode in Tk when we add the noisy ghost shadows? 🙃 Also, I think image12.png is not needed.>

<img width="77" alt="image" src="https://user-images.githubusercontent.com/216179/85016852-d962c500-b16a-11ea-93d8-9e85d890a651.png">>

-- > You are receiving this because you authored the thread.> Reply to this email directly or view it on GitHub:> https://github.com/ASPP/pelita_template/pull/38#issuecomment-645967463

Debilski commented 4 years ago

I started implementing it but now realised that it probably doesn’t work very well. The UI shows where a bot moved but the noise would depend on the previous location. To do it properly we’d need more info in the UI’s game state. Let’s postpone this whole thing, sorry. (Either way it would be nice though, to have a tool to autogenerate these images.)