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

[MIRROR] Removes miasma and gibs diseases #2675

Merged
merged 2 commits into from
May 26, 2024
Merged

Conversation

NovaBot13
Copy link
Collaborator

Original PR: tgstation/tgstation#83455

About The Pull Request

Removes miasma generating random viruses (in two locations for some reason)
Also removes diseases being generated by gibs, which is responsible for a lot of disease spam

Why It's Good For The Game

New diseases are spread through events and virology (and some other minor sources). Miasma and gibs are a source of random viruses that dilutes player response to diseases. It's frankly exhausting curing the disease outbreak, just to see 10 others pop-up. Hopefully we can reduce the madness a bit and place more focus on individual diseases

Changelog

馃啈
del: Miasma and gibs no longer generates random diseases
/:cl:

Something like lung rot could be cool though, but out of scope for the current PR

Time-Green and others added 2 commits May 26, 2024 18:56
## About The Pull Request
Removes miasma generating random viruses (in two locations for some
reason)
Also removes diseases being generated by gibs, which is responsible for
a lot of disease spam

## Why It's Good For The Game
New diseases are spread through events and virology (and some other
minor sources). Miasma and gibs are a source of random viruses that
dilutes player response to diseases. It's frankly exhausting curing the
disease outbreak, just to see 10 others pop-up. Hopefully we can reduce
the madness a bit and place more focus on individual diseases

## Changelog
:cl:
del: Miasma and gibs no longer generates random diseases
/:cl:

Something like lung rot could be cool though, but out of scope for the
current PR
@NovaBot13 NovaBot13 added the Removal This was too fun, too fun! I'm turning this feature around label May 26, 2024
Copy link
Contributor

@FlufflesTheDog FlufflesTheDog left a comment

Choose a reason for hiding this comment

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

Couple upstream oddities but it seems mergable, nothing that needs changing in our files

@@ -45,8 +45,6 @@
RegisterSignal(parent, COMSIG_GLASS_DRANK, PROC_REF(try_infect_drink))
if(isorgan(parent))
RegisterSignal(parent, COMSIG_ORGAN_IMPLANTED, PROC_REF(on_organ_insertion))
else if(istype(parent, /obj/effect/decal/cleanable/blood/gibs))
RegisterSignal(parent, COMSIG_GIBS_STREAK, PROC_REF(try_infect_streak))
Copy link
Contributor

Choose a reason for hiding this comment

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

try_infect_streak is just dead code, now

@@ -111,16 +111,13 @@
drydesc = "They look bloody and gruesome while some terrible smell fills the air."
decal_reagent = /datum/reagent/consumable/liquidgibs
reagent_amount = 5
///Information about the diseases our streaking spawns
var/list/streak_diseases

/obj/effect/decal/cleanable/blood/gibs/Initialize(mapload, list/datum/disease/diseases)
. = ..()
AddElement(/datum/element/squish_sound)
RegisterSignal(src, COMSIG_MOVABLE_PIPE_EJECTING, PROC_REF(on_pipe_eject))

/obj/effect/decal/cleanable/blood/gibs/Destroy()
Copy link
Contributor

Choose a reason for hiding this comment

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

no reason for this to exist anymore

Copy link
Contributor

Choose a reason for hiding this comment

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

The gibs/destroy() proc, I mean. It's just a return ..() with nothing else.

@@ -170,7 +166,7 @@
SIGNAL_HANDLER
if(NeverShouldHaveComeHere(loc))
return
new /obj/effect/decal/cleanable/blood/splatter(loc, streak_diseases)
Copy link
Contributor

Choose a reason for hiding this comment

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

blood_splatter = new /obj/effect/decal/cleanable/blood/splatter(src, diseases)

seems like this wasn't fully cleaned up

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Removal This was too fun, too fun! I'm turning this feature around
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants