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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Medical Rework P. 1 - Defib And Death Timer Rework #2806

Closed
wants to merge 6 commits into from
Closed

Medical Rework P. 1 - Defib And Death Timer Rework #2806

wants to merge 6 commits into from

Conversation

carlarctg
Copy link
Contributor

@carlarctg carlarctg commented Mar 10, 2023

About the pull request

Completely reworks how corpses slowly rot away and become permanently dead. Instead of corpses suddenly becoming unrevivable at 5 minutes, they will slowly accrue heart damage for 7 minutes until heartbreak.

However, they are still technically revivable for three more minutes. This is because now you will also accrue 'integrity' damage over time, which cannot be fixed on the colony and requires one to go up to get surgery on the ship. 'Integrity' damage is taken over 9 minutes until it destroys the heart. When this happens, the brain will stop recieving oxygen and is on a one-minute-timer until permanent, irretrievable death.

Corpses can be 'frozen' with stasis bags or cryogenic pods to prevent further decay.

Colonial surgery tables have been replaced with a new, weaker version that can do almost everything but fix organ integrity and brain surgery. They can still fix normal organ damage.

If you have more organ integrity damage than heart damage, the organ integrity value will be used to calculate symptoms of heart damage. This can never cause you to die or fall unconscious by itself if you're alive.

Removed the unused variable 'tod'.

Removed all check_tod() procs and replaced them with is_revivable()

Refactored a lot of code around organs, though it's still repugnant.

Added a new property, narcoleptic. It reduces metabolization rate by 33% if resting/stunned, and by 66% if sleeping/unconscious, the value is improved with levels.

Peridaxon now metabolizes at 1 unit per 20 seconds, instead of the original 1 unit per 40. It now has the narcoleptic property at level 1, meaning to get the most out of peridaxon you must be incapacitated or intentionally resting.

JS is broken for now

Explain why it's good for the game

Completely reworks how corpses slowly rot away and become permanently dead. Instead of corpses suddenly becoming unrevivable at 5 minutes, they will slowly accrue heart damage for 7 minutes until heartbreak.

This is something that is meant to do several things at once.

  1. It allows marines to stay in the round longer.
    Instead of becoming perma-dead at 5 minutes and being unable to ever play the game as a marine again unless you manage to get a faceless rando cryorine spawn, this increases the defib timer by roughly 40%. This will allow marines to be able to play the game more rather than just ending up as skulls on one mistake. In exchange...
  2. It encourages and reinforces shipside medbay by making it vital to operations by curing integrity loss.
  3. It encourages groundside medbay, even, by allowing ground surgeons to reduce, if not cure, the damage caused by heart decay.

It's a three-pronged feature that is intended to help out all sectors of play by allowing marines to play for longer at the cost of needing more downtime to mend their wounds. This should be bundled with improvements to dropship flight time.

However, they are still technically revivable for three more minutes. This is because now you will also accrue 'integrity' damage over time, which cannot be fixed on the colony and requires one to go up to get surgery on the ship. 'Integrity' damage is taken over 9 minutes until it destroys the heart. When this happens, the brain will stop recieving oxygen and is on a one-minute-timer until permanent, irretrievable death.

This means that technically, if you rush medevac into surgery, you can save someone who is heartbroken from damage over time via replacing their heart (We can still do that right?), which could be a fun little coop minigame if the stars align for it.

Additionally, it means that you actually die from brain death, not your heart failing.

Corpses can be 'frozen' with stasis bags or cryogenic pods to prevent further decay.

This will further increase effective revival time to help marines live longer, at the cost of everything else stated elsewhere.

A lot of things need tuning here. Dropship timing, the exact numbers and breakthrough values, etc. There are surely many values here that could be improved for a better experience.

Colonial surgery tables have been replaced with a new, weaker version that can do almost everything but fix organ integrity and brain surgery. They can still fix normal organ damage.

Again, to encourage going up shipside rather than using the dirty, dusty colony's operating theatres that are filled with blood, grime, acid, and resin. They're still great for anything else though.

If you have more organ integrity damage than heart damage, the organ integrity value will be used to calculate symptoms of heart damage. This can never cause you to die or fall unconscious by itself if you're alive.

This is so that fixing your heart doesn't mean that you're perfectly fine until organ integrity suddenly makes you perma-dead. There's still penalties!

Added a new property, narcoleptic. It reduces metabolization rate by 33% if resting/stunned, and by 66% if sleeping/unconscious, the value is improved with levels.

Peridaxon now metabolizes at 1 unit per 20 seconds, instead of the original 1 unit per 40. It now has the narcoleptic property at level 1, meaning to get the most out of peridaxon you must be incapacitated or intentionally resting.

With Peridaxon just being less useful, marines would be less eager to use it to stay in the fight, and rather use it to retreat to a place they can get healed up. By making it still slow-metabolizing if the marine is resting, the rate of metabolization will hopefully not impact patients but just those who are up and about. It is still usable for those who wish to do this, but they won't be able to fill a canteen and be done for the entire OP.

Also a new shitty toy for researchers, their favorite. Enjoy!

Testing Photographs and Procedure

Screenshots & Videos

Put screenshots and videos here with an empty line between the screenshots and the <details> tags.

Changelog

馃啈
balance: Completely reworks how corpses slowly rot away and become permanently dead. Instead of corpses suddenly becoming unrevivable at 5 minutes, they will slowly accrue heart damage for 7 minutes until heartbreak.
balance: However, they are still technically revivable for three more minutes. This is because now you will also accrue 'integrity' damage over time, which cannot be fixed on the colony and requires one to go up to get surgery on the ship. 'Integrity' damage is taken over 9 minutes until it destroys the heart. When this happens, the brain will stop recieving oxygen and is on a one-minute-timer until permanent, irretrievable death.
balance: Corpses can be 'frozen' with stasis bags or cryogenic pods to prevent further decay.
balance: Colonial surgery tables have been replaced with a new, weaker version that can do almost everything but fix organ integrity and brain surgery. They can still fix normal organ damage.
balance: If you have more organ integrity damage than heart damage, the organ integrity value will be used to calculate symptoms of heart damage. This can never cause you to die or fall unconscious by itself if you're alive.
code: Removed the unused variable 'tod'.
code: Removed all check_tod() procs and replaced them with is_revivable()
refactor: Refactored a lot of code around organs, though it's still repugnant.
balance: Added a new property, narcoleptic. It reduces metabolization rate by 33% if resting/stunned, and by 66% if sleeping/unconscious, the value is improved with levels.
balance: Peridaxon now metabolizes at 1 unit per 20 seconds, instead of the original 1 unit per 40. It now has the narcoleptic property at level 1, meaning to get the most out of peridaxon you must be incapacitated or intentionally resting.
/:cl:

@github-actions
Copy link
Contributor

You currently have a negative Fix/Feature pull request delta of -6. Maintainers may close this PR at will. Fixing issues or improving the codebase will improve this score.

@carlarctg
Copy link
Contributor Author

Peridaxon and stasis are untested

@github-actions github-actions bot added Mapping did you remember to save in tgm format? UI deletes nanoui/html Balance You need to be a professional veteran game maintainer to comprehend what is being done here. Code Improvement Make the code longer Refactor Make the code harder to read labels Mar 10, 2023
code/modules/mob/living/carbon/human/death.dm Outdated Show resolved Hide resolved
code/modules/organs/organ_internal.dm Outdated Show resolved Hide resolved
code/modules/organs/organ_internal.dm Outdated Show resolved Hide resolved
@harryob harryob marked this pull request as draft March 13, 2023 14:15
@carlarctg carlarctg marked this pull request as ready for review March 13, 2023 23:34
@Herotitan772

This comment was marked as off-topic.

@cm13live
Copy link

This pull request has been mentioned on CM-SS13. There might be relevant details there:

https://forum.cm-ss13.com/t/2806-medical-reowrk-p-1-defib-and-death-timer-rework/814/1

@Segrain Segrain requested review from Segrain and removed request for stanalbatross March 18, 2023 14:23
@github-actions github-actions bot added the Merge Conflict PR can't be merged because it touched too much code label Mar 19, 2023
@carlarctg carlarctg requested review from a team as code owners March 20, 2023 10:17
@github-actions github-actions bot removed the Merge Conflict PR can't be merged because it touched too much code label Mar 20, 2023
@harryob
Copy link
Member

harryob commented Mar 20, 2023

fix ur pops bro

@harryob harryob marked this pull request as draft March 20, 2023 19:32
@morrowwolf morrowwolf added the Do Not Merge If you merge this PR, I will annihilate you label Mar 21, 2023
Copy link
Contributor

@morrowwolf morrowwolf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Heads up, I'm gonna be getting my medical changes up and running soonish. This PR will be behind that one for review.

@github-actions
Copy link
Contributor

This PR has been inactive for long enough to be automatically marked as stale. This means it is at risk of being auto closed in ~ 7 days, please address any outstanding review items and ensure your PR is finished, if these are all true and you are auto-staled anyway, you need to actively ask maintainers if your PR will be merged. Once you have done any of the previous actions then you should request a maintainer remove the stale label on your PR, to reset the stale timer. If you feel no maintainer will respond in that time, you may wish to close this PR youself, while you seek maintainer comment, as you will then be able to reopen the PR yourself

@github-actions github-actions bot added the Stale beg a maintainer to review your PR label Mar 29, 2023
@github-actions github-actions bot closed this Apr 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Balance You need to be a professional veteran game maintainer to comprehend what is being done here. Code Improvement Make the code longer Do Not Merge If you merge this PR, I will annihilate you Mapping did you remember to save in tgm format? Refactor Make the code harder to read Stale beg a maintainer to review your PR UI deletes nanoui/html
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants