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

Crit Rework #10876

Closed
wants to merge 26 commits into from
Closed

Conversation

Fox-McCloud
Copy link
Member

@Fox-McCloud Fox-McCloud commented Feb 11, 2019

This is basically Goon's crit system, so all credit goes to Cogwerks, who was the primary designer around this.

In any event, I've wanted to do this for a long time. Our crit system isn't terribly fun or engaging. It's just kinda a case of you become really slow, then you lapse into hard crit and stare at a black screen until death. It's not terribly interesting for doctors, either, as treating someone who comes in tends to just be "treat underlying damage and send them on their way".

Either case, this crit system will completely turn that on its head. When you hit 0 health, your vision will get blurry, you'll get lots of scary message, and you'll start rolling for a chance to lapse into shock. Your breathing will also become irregular. Shock is treated with saline solution.

If shock is left untreated, you will then acquire the state of cardiac emergency. Here, it becomes even more difficult to breathe, and, if left untreated, your heart will stop. You can treat cardiac emergencies with either epinephrine or atropine.

A stopped heart requires a defib. Not shockingly (no pun intended), defibs are no longer tools for revival; they exist purely to jumpstart hearts. Stopped hearts behave like heart-attacks (paralysis, rapid O2 damage, and brain damage).

On that note, how do you die, you might ask? Well, there's no set point, in terms of overall damage. If you have 200 total damage, you're going to black out until you die/are saved, but otherwise, the point at which you die will either be when brain dies or randomly, weighted towards how much damage you currently have accumulated.

By now, there's a few of you who are wondering how this impacts other mobs, or even some species. Things like xenos, simple animals, and the likes all die like they did before. They accumulate so much damage, they lapse into hard crit, accumulate O2, then die (or in the case of simple mobs; they just die).

Some species do not work well with this new system, at all--as it's heavily dependent on organs and breathing. As such, some species will utilize the old crit system, lest they become unkillable. Currently Slimes, IPCs, Plasmamen and Diona die at specific thresholds (Antag/special species like shadow/shadowlings, skeletons, golems, and abductosr all die at thresholds as well). Of course, some of these species could lose their trait of no breathing and possess a heart, thus allowing them to be put on this new system, but that's kinda taking variety away...and it would really only work for Diona and Slimes

Other changes

  • Refactored the defib so that borg defibs are a subtype as opposed to horrifyingly snowflaked object

🆑 Fox McCloud
add: Crit system completely rehashed; treating crit now requires multiple drug types and is strongly dependent on brain damage for death
del: Defibs no longer revive people
tweak: Corazone recipe removed (reagent remains)
/:cl:

@Fox-McCloud Fox-McCloud added Feature This PR is a new addition to the game Balance This PR will modify how effective something is or isnt labels Feb 11, 2019
@TDSSS
Copy link
Contributor

TDSSS commented Feb 11, 2019

While I am not opposed to this change in principle, is now really the time to heap more changes/relearning on medbay when we have like a bazillion revival/medbay change PRs coming?

Also, does this remove the use of painkillers like hydrocodone, morphine, etc. of preventing pain crit, or do they work to prevent the new shock state?

@Fox-McCloud
Copy link
Member Author

Paincrit is done. Paincrit was part of the old soft crit system.

@MrMagolor
Copy link
Contributor

You would be able to detect shock/failure/arrest with a health analyzer, right?

@Fox-McCloud
Copy link
Member Author

Yes.

@Spartan6
Copy link
Contributor

I’m a fan for trying it out.

@TDSSS
Copy link
Contributor

TDSSS commented Feb 11, 2019

I know paincrit's done. I'm asking if the chems used to treat paincrit will be given a new use to compensate or have just been nerfed.

@Shadeykins
Copy link
Contributor

Neat!

@DesolateG
Copy link
Contributor

DesolateG commented Feb 11, 2019

Can you change the description from a stopped heart to Cardiac Arrithymia?

@granodd
Copy link
Contributor

granodd commented Feb 11, 2019

Vaguely reminds me of Bay's Brainmed system where the life of the patient is ultimately tied to the status of the brain. Is there any way of bringing someone back from braindeath with this system outside of cloning?

@Fox-McCloud
Copy link
Member Author

Strange reagent will bring them back, but they're likely to just die immediately after. Reviving people with strange reagent is going to be particularly difficult with this new system. Very involved, and very touch and go.

@TMR-SS13
Copy link

I don’t have a problem with the crit system, but if literally any other changes besides this go through to SR, it’s going to be a huge fuck you to vox and slime people. No defibs, no cloning, and SR is either removed or really hard to get and scarce. I’m still not a fan of this, because defibs have already been nerfed down to two minutes. That’s actually realistic in terms of how you can restart the stopped heart of a dead person, and I don’t see how removing their reviving functionality entirely is going to help anything.

@KasparoVy
Copy link
Contributor

KasparoVy commented Feb 11, 2019

Holy fuck this is gonna be sick
I'm ready to die, Voxing it up as always
And to think when I started playing years ago, I figured spamming resist kept you alive for longer in hard crit

At least now I get to wander around like that one clip of a dude who somehow walked for a bit after a bomb went off nearby. They definitely still died but I mean, not insta-ded or just laying about!

@Mitchs98
Copy link
Contributor

Mitchs98 commented Feb 11, 2019

Agreeing with Book. This is actually a reasonable PR to give a nice trial run too. Provided no changes to SR are made, given it will now be more necessary than ever and use up pills faster while reviving people. This is actually quite interesting, I'd be willing to see it in action and it could prove a worth while change; again so long as SR isn't touched. At-least...not touching SR at first if you people still insist on 'nerfing' it.

@Fox-McCloud Fox-McCloud added the Do Not Merge This PR must not be merged or closed label Feb 11, 2019
@Fox-McCloud
Copy link
Member Author

Marked as DNM because this will require a bit of livetesting first for potential bugs.

@DarkPyrolord
Copy link
Contributor

As someone who started SS13 on Goon and has been playing a bit of Goon as of late, I will say this is one thing I wish more servers would adopt and glad there is a PR for it here as crit in most cases is just bland to say the least, can't wait to see it tested!

@@ -0,0 +1,115 @@
/datum/disease/shock
Copy link
Contributor

Choose a reason for hiding this comment

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

Could use buff/debuff framework for this? I mean we do have it.

Copy link
Member Author

Choose a reason for hiding this comment

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

the system works better as a disease, as it will show up on medical HUDs. Also just the natural of progression/removal and having it died to reagents and based on stages? It's a perfect shoe-in---I'd have to re-create effectively the same systems for a status effect.

@@ -1651,6 +1651,9 @@ Eyes need to have significantly high darksight to shine unless the mob has the X
..()

/mob/living/carbon/human/proc/do_cpr(mob/living/carbon/human/H)
if(H == src)
to_chat(src, "<span class='warning'>You cannot perform CPR on yourself!</span>")
Copy link
Contributor

Choose a reason for hiding this comment

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

WELL WHY THE HELL NOT?! THIS IS BS SHITAINER REEEEEEEEEEEEEEEE

Copy link
Contributor

Choose a reason for hiding this comment

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

Agreed. Literally unplayable. Changed my mind. Do not merge ever.

Copy link
Contributor

Choose a reason for hiding this comment

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

What do you mean I can't punch myself in the chest or run into a wall with enough force to compress my chest and kickstart my heart

@SpinningSalami
Copy link

If going into shock is a disease will having a triple healing virus affect going into shock in any way since 3 is the max amount of diseases you can have?

@Couls
Copy link
Contributor

Couls commented Feb 11, 2019

Does this play nice with no breathing genes and viruses and implants? I would assume it bypasses these things

@Fox-McCloud
Copy link
Member Author

Fox-McCloud commented Feb 11, 2019

If going into shock is a disease will having a triple healing virus affect going into shock in any way since 3 is the max amount of diseases you can have?

That limits is only for advanced viruses.

Does this play nice with no breathing genes and viruses and implants? I would assume it bypasses these things

Species that have no breathing by default are going to die at a threshold, so they use the old system.

Having the breathless gene from genetics means you're going to still take brute damage from being in crit, still acquire shock, still acquire heart problems, and still roll for brain damage. You'll be a bit more durable, but still killable.

I know paincrit's done. I'm asking if the chems used to treat paincrit will be given a new use to compensate or have just been nerfed.

You're still slowed from having damage on you; the painkillers still reduce the slow from having damage.

@Rurik123
Copy link

Sorry if this has been touched on; I may have missed it.

After reaching 200 damage and blacking out, is the succumb command still an option to ghost safely?

@@ -16,7 +15,7 @@
)

species_traits = list(NO_BLOOD, NO_BREATHE, VIRUSIMMUNE, NOGUNS, NO_EXAMINE)

dies_at_threshold = TRUE
Copy link
Contributor

Choose a reason for hiding this comment

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

I understand the change for other species, but why remove the abductor's heart and not have them use this new system? There's no difference medically between an abductor and a human as far as I know. Is this for balance reasons?

Copy link
Member Author

Choose a reason for hiding this comment

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

They also have NO_BLOOD, which means the heart really has no reason to be there. It doesn't do anything; the flag prevents having a heart attack, which is a major component of the new crit system. They also have NO_BREATHE which is also problematic.

@AffectedArc07
Copy link
Member

I have played three whole rounds of medical with this system in place and I have made my descision

Its a no, from me

@Ozewse
Copy link

Ozewse commented Feb 18, 2019

CMO's hypospray starts with mitocholide as opposed to omnizine.

Only thing I 100% agree with out of all the medbay PRs. This is the only SR change we need.

pulled back cryo tubes a bit to further prevent them from being cure-alls.

Not without fixing my sleepers first. Saline glucose isn't stabilizing patients fast enough. You want to get rid of the specialized chemicals in it, fine. I can see why. It makes surgeon and MD harder, but I'm good at those roles. I argued for hours in the Discord about how the changes to sleepers were redundant because actually shitty doctors use cryo tubes for everything and don't even know what half the things in a sleeper do. That said, being that the previous PR made all of medbay hinge on chemists- This nerf should be able to be compensated for in game with a good cryomix, right?

As this is now reagent-medbay, Fix O Vein now does the repair work for internal organs (as opposed to a bruise pack).

Seems like making the surgery kit smaller, might be a good idea? But bruise packs are easy to come by and work for a bajillion surgeries before they run out anyway. Canon - Bruise packs have a few bandaids and hundreds of little sheets of regenerative membrane in them for patching up organs.

This massive PR, making changes to how everyone dies -and- multiple medbay systems.

Should be broken up into multiple entries for clarity.

@Fox-McCloud
Copy link
Member Author

re-adds the beakers and droppers to the medical vendors

@AffectedArc07
Copy link
Member

Things to note after playing even more rounds of this

  • The random event that nukes the vendors makes medbay useless since it is so reliant on them and you cannot rebuild them
  • Medbay is also far too reliant on the cloner now, like more reliant on the cloner than we used to be on cryotubes
  • The lack of auto injectors in boxes is a REALLY big issue

@Spartan6
Copy link
Contributor

It’s becoming normal now to see multiple cloning tubes. I’ve seen no effective change really, nothing possitive at least, I’m sorry to say.

@Mitchs98
Copy link
Contributor

The Changelog and OP needs to be revived so people have an easier way of seeing what all is changed and do not assume that what's there is the only thing changed. It'll give people an easier way to actually judge the PR as there's a lot of posts on this PR and the changes are spread out/not really well documented.

@farie82
Copy link
Member

farie82 commented Feb 20, 2019

Tried it in action and I don't like it sorry. Just feels... dull... Cloning is the to go to option now. Getting into crit means you can still do stuf but die eventually and can't get back. So short term you can live longer, maybe get out of trouble. But long term you get punished harder. I liked the previous version where you go down when you're almost dead. Last game I played I had to kill all of security who responded to me because they kept standing up with 10 HP left (-90 HP). I dunno it just doesn't make the game better.

@KasparoVy
Copy link
Contributor

KasparoVy commented Feb 20, 2019

I've noticed it's much easier for me to close and walk away from the game 🤔 instead of wondering if I have a chance to survive I'm far more certain of my demise.

I've had to employ the tactic of resting and playing dead more regularly because my character gets up and makes people think I'm still a threat (or something to be killed in the case of murderboners who don't finish the job)

I've also avoided medical roles like the plague. I haven't had a good time of it recently but now I feel like the atmosphere is too tense to enjoy that sort of thing.

Others must have caught on, too, as medbay and security population is abnormally low at roundstart during playtests (eg. completely bare ass empty on officers/doctors, only somewhat staffed by latejoiners a ways into the round) but I suppose that's to be expected? I regularly latejoin and avoid rounds with exceedingly poor staffing in those departments and have found this to be necessary, recently, only during newcrit playtests.

I sincerely hope that won't be the case long-term as that is a symptom of reduced enjoyment in that part of the game which in turn negatively effects most others due to the nature of mortality and medbay.

I have mixed feelings and significant apprehension, so my reaction to the parent post has been adjusted accordingly.

@farie82
Copy link
Member

farie82 commented Feb 20, 2019

Small issues with this as addition. The cardiac failure thing is a virus and can be made into a culture. Unsure if you can infect others with the culture but hey.

@Ralta
Copy link
Contributor

Ralta commented Feb 20, 2019

I was very supportive of this initially, but little tweaks and changes along the way has pushed me the other way. As it stands, I don't support the PR in its current state, but feel it would be more viable with the following changes:

  • Oxyloss mute should happen MUCH higher than 10. 50+, imo.
  • Re-add epi pens to emergency boxes. Most people will die, and be unrevivable in any form whatsoever by the time they reach medbay.
  • Defib timer needs to be increased to 5 mins. Derp, forgot about the no-res. Ignore 👀
  • Cloner needs to take a lot longer or require more material.
  • Might need some more chems back in sleeper, but difficult to say exactly which would be useful without reverting sleepers to their previous ridiculous state.
  • Removing advanced medkits means mobs can no longer be healed. Either add a way for this to occur, or re-add advanced medkits (but keep fix-o-vein being used for organ repair - that's a good change).

Please consider some adjustments back towards viability. I'm all for making medbay a little more interesting/challenging, and making crit more nuanced, but the additional changes feel heavy-handed and are ruining it, in my opinion.

@AffectedArc07
Copy link
Member

AffectedArc07 commented Feb 20, 2019

@Ralta

Defib timer needs to be increased to 5 mins.

The timer doesnt even exist. Defibs dont res

@Ralta
Copy link
Contributor

Ralta commented Feb 20, 2019

@Ralta

Defib timer needs to be increased to 5 mins.

The timer doesnt even exist. Defibs dont res

Oh you right. Ignore that bit 👀

@Mitchs98
Copy link
Contributor

  • Cloner needs to take a lot longer or require more material.

I don't exactly agree with this if this is merged. Especially considering how hard it is already to treat newcrit patients. It's nonsensical to make something worse(I.E newcrit) then make something else even worse to compensate(I.E cloning).

If this change does get merged I'm getting up a PR to re-add advanced kits ASAP. Either to medical or as a cargo crate. I have no idea why they were even removed.

@Ralta
Copy link
Contributor

Ralta commented Feb 22, 2019

  • Cloner needs to take a lot longer or require more material.

I don't exactly agree with this if this is merged. Especially considering how hard it is already to treat newcrit patients. It's nonsensical to make something worse(I.E newcrit) then make something else even worse to compensate(I.E cloning).

If this change does get merged I'm getting up a PR to re-add advanced kits ASAP. Either to medical or as a cargo crate. I have no idea why they were even removed.

The reason cloning should take longer is that if this gets merged, cloning suddenly becomes the easiest solution to most problems. Building extra cloners and getting a ton of material would be simpler than fixing more complex medical problems. I think that would be lessened by some of my other suggestions, but I still feel the point stands.

@shazbot194
Copy link
Contributor

The reason cloning should take longer is that if this gets merged, cloning suddenly becomes the easiest solution to most problems. Building extra cloners and getting a ton of material would be simpler than fixing more complex medical problems. I think that would be lessened by some of my other suggestions, but I still feel the point stands.

Isn't that the problem though? These changes make it so it is much much easier to just clone most people rather then trying to fix them. Would nerfing another system to parody a nerf really be necessary? Shouldn't it be the other way around, and not make the system so harsh so people use it?

@runemeds
Copy link

runemeds commented Feb 22, 2019

Shazbot is right. The concept of nerfing a system that was made stronger by the nerfing of a different system is remarkably bad game design, and a poor choice for game health. This goes for all games.

@Terilia
Copy link
Contributor

Terilia commented Feb 22, 2019

I am mixed about the new system - and here is why.

I like the idea of reworking crit, of making it more deadly and more serious, but it comes with a huge nerf to medical systems and makes the decision often just a two way street.

Can I fix him fast? - Fix him fast
Anything more complicated? - Into cloning they go.

This two-way approach to medical is partially due to players being lazy, and partially due to them just not knowing better, the chemist being trash or other unforeseen consequences.

I had a few rounds where Fox was my Doctor, and I have seen in person that, if you know what you are doing, you can fix people without stuffing them into cloning. In addition, several times he fixed me up from things that I thought were a guaranteed death. Nevertheless, unfortunately not every player is like this.

Personally, I would ask for more depth in the medical system. I have just taken a short look at TG station, and frankly, I know nothing about how their medical system works. However, the idea alone of brain damage that add random effects sounds far more advanced than what we have right now. I agree, medical should become more interesting, but personally, I would add more systems into the diagram of procedures. Example: Allow them to clone people, but - it might result in brain damage - and woops, person has now "Monophobia - The patient will become increasingly stressed when not in company of others, triggering panic reactions ranging from sickness to heart attacks." which needs to be treated as well.

In addition: No system should be reliant on one factor. Currently, I feel like the chemist is too important. Destroy the cloner, kill the chemist (or chemist area) – Medbay is done. Back then they still had the defib as a last resort, but this path is now closed.

Overall - the game is Semi Realistic - allowing us to bend the rules, reality, and time just enough. Because if Survival Games taught me anything: Realism is not always fun.

@Aurorablade
Copy link
Contributor

For the phobia/Brain Trauma system we would need some systme ports from TG probaly with thier quirk system. We just need someone to Actually do that..

@Aurorablade
Copy link
Contributor

I am seeing some comments about the current Defib system being too damming though, would there be a compromise to that somewhere?

@AffectedArc07
Copy link
Member

Image
Not 100% sure how related to the PR this is but I am putting it here so it’s seen

@Aurorablade
Copy link
Contributor

That color scheme hurts my eyes. ANYWAY, we did have some feedback saying that someone fox and i am sure anyone else who knows the system..played..and it actually worked. I really do not think its the system at fault here.

@Mitchs98
Copy link
Contributor

That color scheme hurts my eyes. ANYWAY, we did have some feedback saying that someone fox and i am sure anyone else who knows the system..played..and it actually worked. I really do not think its the system at fault here.

Just because it works doesn't mean it's fit for this server.

@Raze500
Copy link

Raze500 commented Mar 1, 2019

Nobody asked for this, but ok.

@AffectedArc07
Copy link
Member

Why doesnt this have the [Controversial] label

@Fox-McCloud Fox-McCloud closed this Mar 6, 2019
@Fox-McCloud Fox-McCloud deleted the crit-rework-again branch January 23, 2021 00:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Balance This PR will modify how effective something is or isnt Do Not Merge This PR must not be merged or closed Feature This PR is a new addition to the game Fix This PR will fix an issue in the game Literally Everybody Ded Were we ever alive? Refactor This PR will clean up the code but have the same ingame outcome Work In Progress This PR is work in progress, and unfinished
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet