search

llaske / sugarizer

Sugarizer is a web implementation of the Sugar platform to run on any device or browser
https://sugarizer.org
Apache License 2.0
197 stars 412 forks source link

Player without face and color #1536

Closed hackorlyf closed 5 months ago

hackorlyf commented 6 months ago

PlayerNotSpawned

hey @llaske there are times I faced this minor bug on maze web activity where the player initially is faceless and of a grey color until and unless we give it an input. This is at total random and I had to restart twice from start to re-create it. I want to work on it.

llaske commented 6 months ago

Can't reproduce it on my side. Try to find the sequence to reproduce it.

hackorlyf commented 6 months ago

Okay I'll look into it.

On Fri, Feb 23, 2024 at 2:20 AM Lionel Laské @.***> wrote:

Can't reproduce it on my side. Try to find the sequence to reproduce it.

— Reply to this email directly, view it on GitHub https://github.com/llaske/sugarizer/issues/1536#issuecomment-1960291444, or unsubscribe https://github.com/notifications/unsubscribe-auth/A7CHCCVCNX47KY7AI263NMTYU6VQ7AVCNFSM6AAAAABDTH4VTGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRQGI4TCNBUGQ . You are receiving this because you authored the thread.Message ID: @.***>

hackorlyf commented 6 months ago

@llaske just keep restarting the maze web activity by the button provided in the activity itself and play upto 7-8 level. It shows at the start state so don't give any input until the whole maze is loaded. You'll be able to reproduce it somewhere in between restarting 3-4 times.

llaske commented 6 months ago

You're right, I've reproduced it

image

hackorlyf commented 6 months ago

I wanna contribute in helping you fix this bug

llaske commented 6 months ago

You're welcome to send a PR

hackorlyf commented 6 months ago

any idea what part of code might be causing this bug?

llaske commented 6 months ago

nope

sonisumit7904 commented 6 months ago

There might be a refress action being performed after every input, because of which after input the face is visible, but maybe sometimes at the start of the game, that refresh action doesn't work, You could use this as a starting point for searching the bug.

hackorlyf commented 6 months ago

Thank you so much @sonisumit7904 Got stuck sometime ago but I'll look further into it.

sonisumit7904 commented 6 months ago

@hackorlyf Happy to help!

UtkarshSiddhpura commented 6 months ago

@hackorlyf Thanks for bringing this issue, i was just looking the issue but then really got into it, as it was pretty hard to debug, and after sometime i finally fixed it, you can verify the working of above solution.

llaske commented 6 months ago

Fixed in https://github.com/llaske/sugarizer/pull/1541