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

Makes the progression of blood-loss more granular #17092

Merged

Conversation

colossusqw
Copy link
Contributor

@colossusqw colossusqw commented Nov 25, 2023

[LABEL][Medical][Balance]

About the PR

This PR adds a chance of drowsiness, stuttering, blurry vision and shiver slowdowns when the player is between 300 and 415 blood.

Why's this needed?

As it stands, the divide between more than 300 blood and less than 300 blood is very binary, and you go pretty fast from very negligible effects to extremely severe ones. Hopefully this PR would make the stage just before the dangerous part of low blood pressure more significant, and make the progression of blood loss more linear and less binary.

Changelog

(u)Colossusqw
(+)Non-lethal bleeding now has a bit more imposing effects.

@github-actions github-actions bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Nov 25, 2023
@keywordlabeler keywordlabeler bot added A-Medical Deals with our medical system in some way C-Balance Balance changes, buffs and nerfs labels Nov 25, 2023
Copy link
Member

@frawhst frawhst left a comment

Choose a reason for hiding this comment

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

lgtm

@colossusqw
Copy link
Contributor Author

Last commit changed 7 speed slowdowns to 2 speed shivers, timings remain the same as regular shiver times
Should be more on par with what's expected from bleeding, if a tad tamer

@goonstation-issuebot goonstation-issuebot removed the S-Testmerged [Dev Only] Testmerged for extended testing (applied by bot) label Dec 6, 2023
@frawhst frawhst merged commit 98eb950 into goonstation:master Dec 6, 2023
23 checks passed
github-actions bot pushed a commit that referenced this pull request Dec 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Medical Deals with our medical system in some way C-Balance Balance changes, buffs and nerfs size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants