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

Adds allergic reactions to chems #8263

Merged
merged 4 commits into from May 5, 2020
Merged

Conversation

ToGWtF
Copy link
Contributor

@ToGWtF ToGWtF commented Apr 25, 2020

Intent of your Pull Request

Adds a fairly benign negative quirk, Allergies. Every chem that you can be allergic to can be replaced with other healing methods round-start, and doesn't actually stop the chem from working in you. Its just a mild annoyance at the moment.

Should I add food allergies as well to the list it can pick?
Should I randomize the amount of histamine it gives?
Maybe add some other effects? Like a "seizure" putting you unconcious or pure toxin

Changelog

馃啈
rscadd: Added a negative quirk that will make you allergic to some chems
/:cl:

@yogstation13-bot yogstation13-bot added the Feature This adds new content to the game label Apr 25, 2020
@Jarod1200
Copy link

will a person's allergies show up on a health analyzer?

@Darkstick
Copy link
Contributor

All annoyance zero benefit. why?>

It's a negative quirk. You either take them for the positive quirk points or just for the fun of it.

@ToGWtF
Copy link
Contributor Author

ToGWtF commented Apr 25, 2020

All annoyance zero benefit. why?>

It's a negative quirk. You either take them for the positive quirk points or just for the fun of it.

Yes, this is mostly for character building rather than actually affecting your ability to play. Considering making it a neutral quirk

will a person's allergies show up on a health analyzer?

I haven't even looked into making that work but I will take a look

code/datums/traits/negative.dm Outdated Show resolved Hide resolved
@overlord12345
Copy link

food allergies seem like a good idea

@Marmio64
Copy link
Contributor

if it doesn't actually affect you game play wise yes it needs to be a neutral quirk.

@Hopekz
Copy link
Contributor

Hopekz commented Apr 25, 2020

Would Synthflesh work?
If so; you should add it.

if it doesn't actually affect you game play wise yes it needs to be a neutral quirk.

An allergic reaction gives you 5 units of histamine in your body and keeps getting this added the more an allergic chem remains in your body.

@ToGWtF
Copy link
Contributor Author

ToGWtF commented Apr 25, 2020

Would Synthflesh work?
If so; you should add it.

if it doesn't actually affect you game play wise yes it needs to be a neutral quirk.

An allergic reaction gives you 5 units of histamine in your body and keeps getting this added the more an allergic chem remains in your body.

with the values i have set the histamine will wear off before the next proc of it, even with 10u histamine. should i rand(3,15) it instead of just 5?

I would add synthflesh as well but I wanted to keep the list fairly low to maximize the potentially of getting something like bicardine/kelotane. With the synthflesh changes to only work on a dead body the histamine would wear off before you woke up from being defibbed anyway.

@Hopekz
Copy link
Contributor

Hopekz commented Apr 25, 2020

With the synthflesh changes to only work on a dead body the histamine would wear off before you woke up from being defibbed anyway.

Who really knows if the Synthflesh changes will get merged; we'll see.

code/datums/traits/negative.dm Outdated Show resolved Hide resolved
code/datums/traits/negative.dm Outdated Show resolved Hide resolved
code/datums/traits/negative.dm Outdated Show resolved Hide resolved
code/datums/traits/negative.dm Outdated Show resolved Hide resolved
code/datums/traits/negative.dm Outdated Show resolved Hide resolved
code/datums/traits/negative.dm Outdated Show resolved Hide resolved
code/datums/traits/negative.dm Outdated Show resolved Hide resolved
@Hopekz Hopekz merged commit d3eab46 into yogstation13:master May 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature This adds new content to the game
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants