Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug] Room explosion process differs from TH #2556

Open
2 tasks
MandrakeHorse opened this issue Apr 14, 2024 · 5 comments
Open
2 tasks

[Bug] Room explosion process differs from TH #2556

MandrakeHorse opened this issue Apr 14, 2024 · 5 comments
Labels
bug issue type S:Help wanted Development needed

Comments

@MandrakeHorse
Copy link

MandrakeHorse commented Apr 14, 2024

Describe the issue

Whenever machinery explodes from either overuse or an earthquake, the room explosion process does not match what happens in Theme Hospital:

  • In CorsixTH, there is no explosion animation and instead, the room instantly becomes destroyed. In Theme Hospital, an explosion can be seen.
  • In CorsixTH, if a patient is present in the room, they are immediately deleted (similar to staff). In Theme Hospital, the patient enters a dying state where they collapse to the ground (possibly with a sprite change), get back up, become an angel and fly upwards to heaven.

Steps to Reproduce

  1. Load save game attached to this ticket
  2. Wait until the Operating Table explodes from overuse

Expected Behaviour

The expected behaviour can be see in this Let's Play: https://youtu.be/VqJeh3i5j1w?t=728

  • The machines will have an explosion animation when destroyed.
  • When a patient is killed by exploding machinery, they will enter a dying state where they collapse to the ground, get back up, become an angel and fly upwards to heaven.

Save Game

Save game attached to report: Yes
level7operatingtableboom.zip

CorsixTH Version

0.67

Operating System

Windows 10

Theme Hospital Version

CD

Gamelog.txt

No response

Additional Information

Lewri: additionally the explode anim being missing I'm merging into this issue (#2553)

@MandrakeHorse MandrakeHorse added the bug issue type label Apr 14, 2024
@lewri
Copy link
Member

lewri commented Apr 14, 2024

They also briefly become a female patient (that's a TH bug though)
image

@lewri
Copy link
Member

lewri commented Apr 14, 2024

There are explosion animations in the TH files for a single cloud, which I guess it spawns in a randomly placed multiple of them on explode. This is probably doable enough, and might be interesting for someone to look at and play with.

The Dr still just instantly despawns (probably because staff have no death anims). Patients currently are told to always die in a corridor. It'd need a code change to let them die in the room during explosions (rather than just despawn)

@lewri lewri added the S:Help wanted Development needed label Apr 14, 2024
@lewri lewri changed the title [Bug] Patient does not become an angel if killed by machinery explosion [Bug] Room explosion process differs from TH Apr 14, 2024
@MandrakeHorse
Copy link
Author

Sidenote: The explosion animation also appears when firing the bomb weapon in the Rat Bonus Level, so it can be reused once we get around to implementing this minigame:
image

@lewri
Copy link
Member

lewri commented Apr 21, 2024

Have we got any idea of how this animation effect is made specifically, and is anyone interested in replicating it to CorsixTH?

@Alberth289346
Copy link
Contributor

Never saw it in action, but the second screenshot looks to me as 2 explosions. The big smoke spheres likely expanded from small spheres before taking the screen shot.
The small spheres are then relatively younger.

This also seems to happen in the first screenshot, assuming spheres eventually get a darker color.

I'd start with a bunch of explosion animations at random height and random horizontal displacement from some center point as a start. Then shortly after it another bunch of such animations.

Depending on how it looks you may want to adjust things, eg reduce displacement with increasing height.

I am not particularly interested in implementing it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug issue type S:Help wanted Development needed
Projects
None yet
Development

No branches or pull requests

3 participants