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

Minor rework to how toxic chemicals work #22504

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

Moltijoe
Copy link
Contributor

@Moltijoe Moltijoe commented Aug 10, 2024

Toxic chemicals now work by
Put the toxpower of every chemical in a list in descending order
first chemical (highest) does full damage
second chemical does 50% reduced damage
third chemical does 75% reduced damage
fourth chemical does 87.5% reduced damage

and so forth and so on

Why is this good for the game?

i hate the arbitrary purge threshold, it makes certain toxins that are designed to be potent in small amounts utterly useless, as they get instantly purged when low
Changing it to this allows for something like cyanide to actually be good in small doses
while still preventing loading a syringe with 1u of 30 different toxins to instantly kill someone

Gave polys a toxmod to account for their livers being nerfed a bit

Testing

image

🆑
tweak: Polysmorph now have a 0.8 toxmod
tweak: Livers no longer instantly purge toxic chemicals when below an arbitrary number
tweak: Toxic chemicals now deal half damage for every stronger toxin in the bloodstream
tweak: Toxic chemicals now no longer scale liver damage with volume
/:cl:

@Moltijoe Moltijoe added the Awaiting - Action - Testing TEST YOUR STUFF label Aug 10, 2024
@Yogbot-13 Yogbot-13 added the Tweak This PR contains changes to the game. label Aug 10, 2024
@Moltijoe Moltijoe removed the Awaiting - Action - Testing TEST YOUR STUFF label Aug 12, 2024
toxpwr += T.toxpwr
provide_pain_message = provide_pain_message || !T.silent_toxin

sort_list(toxpwr, cmp = /proc/cmp_numeric_dsc) //sort all toxpowers scaling from highest to lowest
Copy link
Contributor

@SapphicOverload SapphicOverload Sep 4, 2024

Choose a reason for hiding this comment

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

I feel like this should probably be cached on the liver and updated whenever a toxin is added or removed from the body so you don't have to sort the list every single tick.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

guh, i considered that but it sounds like a lot of work

@Moltijoe
Copy link
Contributor Author

i really should implement sarah's suggestion, but i don't have the motivation, and haven't for a while

@Moltijoe Moltijoe closed this Sep 25, 2024
@Moltijoe Moltijoe deleted the ToxinChemicalsRework branch September 25, 2024 18:56
@Moltijoe Moltijoe restored the ToxinChemicalsRework branch October 2, 2024 22:22
@Moltijoe Moltijoe reopened this Oct 2, 2024
@JamieD1
Copy link
Contributor

JamieD1 commented Nov 21, 2024

How we looking @Moltijoe

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Tweak This PR contains changes to the game.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants