med-material / Whack_A_Mole_VR

Whack-A-Mole in VR
MIT License
3 stars 16 forks source link

Randomly Mole Spawner System #210

Closed Xav1204 closed 1 year ago

Xav1204 commented 2 years ago

We needed to add a small animation on the wall to make the experience more immersive. Now, the mole spawn randomly on the wall surface.

What we still need to do: - Add a fadeout transition for the countdown panel

Xav1204 commented 2 years ago

Animation.zip

I made a gif to show the animation but it was too heavy so I compressed it.

hendrikknoche commented 2 years ago

great. For future submissions try to e.g. reduce the resolution because the steps of downloading and unzipping defeat the purpose of having a visual here for easy browsing. :-)

Xav1204 commented 2 years ago

MoleSpawner I did it again. I think that it will be better now.

bastianilso commented 2 years ago

Hi @Xav1204, can you try speeding it up by 2x (double speed)? Just want to see how it looks, compared to current animation you uploaded. thanks!

Xav1204 commented 2 years ago

MoleSpawner @bastianilso Is it enough fast for you?

Xav1204 commented 2 years ago

@bastianilso I pushed the updates. I also commented the code with the explanations of my choices.

bastianilso commented 2 years ago

@Xav1204 Could you please respond directly to my questions in the pull request itself?

image

I think it's fine with the code comments too. By convention we either use code comments if a) we want to document what a function does, or b) if we introduce a hack/do something unconventional/do a workaround of some kind.

Xav1204 commented 2 years ago

@bastianilso I didn't see any questions in your last comment. Do you want that I respond to the question about the interest of the Chunked Lists?

Xav1204 commented 1 year ago

@bastianilso it's also good for the mole spawn animation. We just need to see the behavior of the animation after having solved the conflicts