Skip to content
This repository was archived by the owner on May 22, 2025. It is now read-only.

Conversation

@ToasterBiome
Copy link
Contributor

@ToasterBiome ToasterBiome commented Nov 24, 2021

Document the changes in your pull request

Adds tumors to virology.

I am aware that this bounty is not approved yet (nor do I have any control if it will be approved or not)
https://forums.yogstation.net/threads/30-1-4-add-tumors-as-a-virology-symptom-benign-premalignant-malignant.24324/

Benign, Premalignant, and Malignant tumors.
Benign, Premalignant, and Malignant virology symptoms.

if you have a disease with the tumor symptom, you will have a chance to get a tumor organ in a random slot in your body. Using a medical scanner will tell you where the tumor is.
if you have tumors, you have a chance to have them spread to other areas of your body.
if you remove all the tumors associated with a disease, you get cured.
Benign tumors are mostly harmless, the higher versions are stronger and spread faster.

Transmission 7 makes it so you have visible growths on your body, these will progressively make it so you can wear less and less things depending on stages: stage 3 you lose the ability to wear hats/masks. stage 5 you cant wear chest slot items.
Stealth 4 makes it so you have a chance to regenerate limbs
Resistance 8 makes you heal brute/burn damage for each tumor you have.

Benign Tumors:
0.125 toxic damage per life tick
if sufficient resistance, heal 0.0625 brute and 0.0625 burn per life tick
0.25% chance to spread per life tick

Premalignant Tumors:
0.25 toxic damage per life tick
if sufficient resistance, heal 0.125 brute and 0.125 burn per life tick
0.5% chance to spread per life tick

Malignant Tumors:
0.5 toxic damage per life tick
if sufficient resistance, heal 0.25 brute and 0.25 burn per life tick
1% chance to spread per life tick

I have no clue if this is balanced and I need feedback

need to still be implemented:

  • Tumor Sprites (just uses heart sprite for now)
  • Nicer tumor names (just says the slot, so right arm is "r_arm")
  • Limb regeneration
  • visible growths locking you out of clothing slots

Wiki Documentation

3 new virology symptoms, 3 new organs, something about removing them all curing a disease

Changelog

🆑
rscadd: Adds 3 new tumors
rscadd: Adds 3 new virology symptoms
tweak: lets organs be inserted into special funny slots
/:cl:

@Yogbot-13 Yogbot-13 added DME Edit This PR affects the yogstation.DME file Feature This adds new content to the game Tweak This PR contains changes to the game. labels Nov 24, 2021
@Mqiib
Copy link
Contributor

Mqiib commented Nov 24, 2021

So, assuming I'm reading this right, you'll get exponential growth of tumors because each tumor has its own spread proc. At 0.06 tumors per tumor per two seconds we can get a spicy equation set up for how many tumors (spoilers: it's a lot) you can get over the course of your time left (not very long).
image
The red line is the number of tumors, green line is total toxin damage taken by the tumors, x is seconds.
As you can see, about 88 seconds after the first tumor (roughly since we're dealing with single tumors and not fractions of tumors) you'll be in crit from toxin damage AND have about 12 more buddies with you. At around the 109 second mark, you're dead from toxin damage and have 24 or so friends. If you're NOT dead at this point you're taking an exponentially increasing 12 toxin damage per second (that's six times what charcoal heals). If you somehow survive for a whole two minutes, you'll have 33 friends to be surgically removed dealing a combined 16.5 damage per cycle.
But what about the baby tumors? Certainly they're fine?
image
Not really. Instead of being in crit at 88 seconds, you'll stay up for another 200 seconds as you slowly begin to realize what's going on. Death still follows shortly afterward at the roughly 355 second mark, about 70 seconds later. At this point you'll have around 31 friends coming to your funeral.

This needs some limits to how many tumors you can have and a SIGNIFICANT decrease in how fast they grow.

@ToasterBiome
Copy link
Contributor Author

ToasterBiome commented Nov 24, 2021

you can only have 8 tumors

you only get 8 body parts where tumors can grow, and tumors wont grow in places that already have tumors in them

@Mqiib
Copy link
Contributor

Mqiib commented Nov 24, 2021

That still leaves only about 70 seconds to get all 8 malignant tumors and 4 toxin damage per cycle perpetually until you get EIGHT surgeries done to fix it.

@ToasterBiome
Copy link
Contributor Author

ToasterBiome commented Nov 24, 2021

That still leaves only about 70 seconds to get all 8 malignant tumors and 4 toxin damage per cycle perpetually until you get EIGHT surgeries done to fix it.

what would it look like if all the spread times were divided by 2? 4? 8?

@Mqiib
Copy link
Contributor

Mqiib commented Nov 24, 2021

That still leaves only about 70 seconds to get all 8 malignant tumors and 4 toxin damage per cycle perpetually until you get EIGHT surgeries done to fix it.

what would it look like if all the spread times were divided by 2? 4? 8?

half growth speed = double time needed to hit max

@ToasterBiome
Copy link
Contributor Author

That still leaves only about 70 seconds to get all 8 malignant tumors and 4 toxin damage per cycle perpetually until you get EIGHT surgeries done to fix it.

what would it look like if all the spread times were divided by 2? 4? 8?

half growth speed = double time needed to hit max

oh also if it picks that zone to put a tumor into and you already have a tumor it just... doesnt put one in that life tick. does that change the calculations

@Mqiib
Copy link
Contributor

Mqiib commented Nov 24, 2021

yes that would make it complicated enough that i dont want to go through them again but it would also effectively reduce the growth speed

@adamsong adamsong added the DO NOT MERGE Should not be merged without express approval from a Head Dev label Nov 29, 2021
@ToasterBiome
Copy link
Contributor Author

@adamsong sanity review plz????

@ToasterBiome ToasterBiome requested a review from a team as a code owner December 13, 2021 02:34
@github-actions github-actions bot added the Sprites This PR has spriting changes label Dec 13, 2021
@ToasterBiome
Copy link
Contributor Author

@adamsong remove do not merge tag please it is ready

@JamieD1 JamieD1 added Awaiting - Action - Maintainer This PR is awaiting an action from a maintainer and removed DO NOT MERGE Should not be merged without express approval from a Head Dev labels Dec 21, 2021
Copy link
Contributor

@adamsong adamsong left a comment

Choose a reason for hiding this comment

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

Seems to look good code wise, balance is a different concern though

@adamsong adamsong added Awaiting - Action - HD Awaiting action from Jamie and removed Awaiting - Action - Maintainer This PR is awaiting an action from a maintainer labels Dec 22, 2021
@SomeguyManperson
Copy link
Contributor

what exactly is the purpose of this symptom is it supposed to be helpful or harmful? Symptoms like RPS which can kill you if not treated properly feel like noob bait and just get shoved into healing viruses because they "look cool" without any way to actually manage them, tumors would require you to constantly have access to some kind of toxin healing chem or other system to be useful or it'll kill you which makes it seem more wack to me than stuff like plasma fixation which require access to x chemical but WON'T kill you if you don't have it
also use add_no_equip_slot and remove_no_equip_slot procs to reduce the amount of bullshit that might show up from pre-existing no_equip flags

@ToasterBiome
Copy link
Contributor Author

what exactly is the purpose of this symptom is it supposed to be helpful or harmful? Symptoms like RPS which can kill you if not treated properly feel like noob bait and just get shoved into healing viruses because they "look cool" without any way to actually manage them, tumors would require you to constantly have access to some kind of toxin healing chem or other system to be useful or it'll kill you which makes it seem more wack to me than stuff like plasma fixation which require access to x chemical but WON'T kill you if you don't have it also use add_no_equip_slot and remove_no_equip_slot procs to reduce the amount of bullshit that might show up from pre-existing no_equip flags

threshholded RPS deals the same amount of toxin damage in the same amount of time as four of the worst tier of tumors. I think they're supposed to be harmful, but if you get the threshholds get some benefits like the healing, and limb regeneration. So.. harmful until it's helpful?

@ToasterBiome
Copy link
Contributor Author

@SomeguyManperson More feedback on this would be nice to get this to a state you're happy with, it's been testmerged for a month with no issues.

ToasterBiome and others added 6 commits February 15, 2022 18:31
Co-authored-by: TheGamerdk <5618080+TheGamerdk@users.noreply.github.com>
Co-authored-by: TheGamerdk <5618080+TheGamerdk@users.noreply.github.com>
Co-authored-by: TheGamerdk <5618080+TheGamerdk@users.noreply.github.com>
@JamieD1
Copy link
Contributor

JamieD1 commented Mar 7, 2022

Please Resolve your conversations/suggestions when you have done them

@ToasterBiome
Copy link
Contributor Author

Please Resolve your conversations/suggestions when you have done them

@TheGamerdk please reply to my responses so I may move this PR forward

@ToasterBiome ToasterBiome requested a review from Bibby0110 March 18, 2022 02:13
@ToasterBiome ToasterBiome requested a review from JamieD1 March 18, 2022 02:15
@Bibby0110 Bibby0110 added Stale I'm closing this in like a week if you don't push a new commit and removed Stale I'm closing this in like a week if you don't push a new commit labels Mar 18, 2022
@Bibby0110
Copy link
Contributor

Bibby0110 commented Mar 19, 2022

Please Resolve your conversations/suggestions when you have done them

@TheGamerdk please reply to my responses so I may move this PR forward

dont resolve unresolved stuff pls

@Bibby0110
Copy link
Contributor

oh you actually did it, just ignore me

@JamieD1 JamieD1 merged commit 4844ecb into yogstation13:master Mar 27, 2022
Yogbot-13 added a commit that referenced this pull request Mar 27, 2022
@ToasterBiome ToasterBiome deleted the tumor branch April 16, 2024 21:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Awaiting - Action - HD Awaiting action from Jamie DME Edit This PR affects the yogstation.DME file Feature This adds new content to the game Sprites This PR has spriting changes Tweak This PR contains changes to the game.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants