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

Choking prevents the inhalation of smoke or smoking reactions #15909

Merged

Conversation

M-Earthfire
Copy link
Contributor

@M-Earthfire M-Earthfire commented Sep 20, 2023

[CHEMISTRY][INPUT WANTED][BALANCE]

About the PR

This PR prevents inhaling smoked chemicals while losebreath is higher than 0. This means smoke won't be inhaled when you cannot breath due to choking from chems/strangulation.
It applies to reactions that makes you inhale without a gas mask, like meth, as well.

Keep in mind this does not apply to asphixiation due to spaced enviroment.

Why's this needed?

It kinda makes no sense that you are to breath in smoke while actively choking and struggling to breath. This mostly affects chemicals which makes you struggling to breath e.g. cyanide. Also, this makes dying people in smoke not continue to suck of chemicals out of the smoke.

This will make perfluorodecalin able to make you walk through non-skinpen smoke without problems, but since gas masks are fairly aquireable, this shouldn't be much of a problem. It enables perfluorodecalin to be an emergency tool for sudden smoking, though.

This PR enables additional mechanics and design space, like improving what stuff like what #15908 tries to achieve.

Changelog

(u)Lord_Earthfire
(+)Choking makes you unable to breath in chemsmoke.

@keywordlabeler keywordlabeler bot added A-Chemistry Deals with the chemistry system in some way C-Balance Balance changes, buffs and nerfs labels Sep 20, 2023
@github-actions github-actions bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Sep 20, 2023
@M-Earthfire M-Earthfire changed the title Choking prevents the inhalation of smoke Choking prevents the inhalation of smoke or smoking reactions Sep 20, 2023
@keywordlabeler keywordlabeler bot added the E-Input-Wanted Input and feedback are wanted. Also posts a discussion thread on the forums. label Sep 20, 2023
@github-actions
Copy link
Contributor

Created discussion thread: https://forum.ss13.co/showthread.php?tid=21806

@github-actions github-actions bot added the S-Ready-For-Final-Review This PR has been looked over by members of the community. (automatic after 2 approvals) label Oct 1, 2023
@github-actions
Copy link
Contributor

This PR has been inactive for two weeks, and has been automatically marked as stale. This means it is at risk of being auto closed in another week. Please address any outstanding review items and ensure your PR is finished. If you are auto-staled anyway, ask developers if your PR will be merged. Once you have done any of the previous actions then you should request a developer remove the stale label on your PR, to reset the stale timer. If you feel no developer will respond in that time, you may wish to close this PR youself, while you seek developer comment, as you will then be able to reopen the PR yourself.

@github-actions github-actions bot added the S-Stale An inactive PR that has had no updates in the past two weeks label Oct 16, 2023
@Tarmunora Tarmunora merged commit 2eaf22b into goonstation:master Oct 16, 2023
30 checks passed
@M-Earthfire M-Earthfire deleted the no_smoke_breath_while_choking branch October 16, 2023 15:55
github-actions bot pushed a commit that referenced this pull request Oct 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Chemistry Deals with the chemistry system in some way C-Balance Balance changes, buffs and nerfs E-Input-Wanted Input and feedback are wanted. Also posts a discussion thread on the forums. S-Ready-For-Final-Review This PR has been looked over by members of the community. (automatic after 2 approvals) S-Stale An inactive PR that has had no updates in the past two weeks size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants