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

Make Height Sliders Affect Your Bloodstream Volume #858

Merged

Conversation

Mnemotechnician
Copy link
Contributor

@Mnemotechnician Mnemotechnician commented Sep 4, 2024

Description

Something that just makes sense, this makes your effect character weight affect your bloodstream volume. As a minimum size felinid you will get 33% of normal blood volume, whereas as something as huge as a lamia you may get up to 3 times the normal blood volume.

The resulting volume of your bloodstream can be calculated as V = clamp(normal_volume * mass_contest ^ 0.6) (assuming default parameters), where mass_contest is the result of a mass contest between your entity and the average humanoid. For average species like vulps, this means that their bloodstream can become up to ~40% smaller than normal (at minimum size), or up to 50% larger than normal (at maximum size). For onis the range is shifted towards higher values, a maximum size oni will have twice as much blood as an average human.

This has both drawbacks and advantages. For instance, having little blood means you can bleed out easily, but at the same time it means it will take way less blood packs/saline/iron/proteins to restore your blood to the normal level. Opposite is also true, having more blood means you will be harder to heal.

Also, this PR slightly refactors the HeightAdjustSystem to be more flexible.

Media

8mb.video-ZJZ-xcwgAzX7.mp4

Changelog

🆑

  • add: Your character size now affects your blood level. Smaller characters will have less blood, and larger characters will have more.

@github-actions github-actions bot added Changes: C# Changes any cs files Changes: YML Changes any yml files labels Sep 4, 2024
@SimpleStation14 SimpleStation14 changed the title Make height sliders affect your bloodstream volume Make Height Sliders Affect Your Bloodstream Volume Sep 4, 2024
Copy link
Member

@VMSolidus VMSolidus left a comment

Choose a reason for hiding this comment

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

Marking this PR as "Do Not Merge" until #814 is merged, due to this PR depending on an "Unsafe" implementation of Contests, that will require certain safety checks be included in the Contests System.

@VMSolidus VMSolidus added the Status: Do Not Merge Do not merge label Sep 5, 2024
Co-authored-by: VMSolidus <[email protected]>
Signed-off-by: Mnemotechnican <[email protected]>
@github-actions github-actions bot added the Status: Needs Review Someone please review this label Sep 6, 2024
Copy link
Member

@VMSolidus VMSolidus left a comment

Choose a reason for hiding this comment

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

Eh fuck it.

@DEATHB4DEFEAT DEATHB4DEFEAT removed the Status: Do Not Merge Do not merge label Sep 8, 2024
@VMSolidus VMSolidus merged commit 9dd569d into Simple-Station:master Sep 8, 2024
11 checks passed
SimpleStation14 added a commit that referenced this pull request Sep 8, 2024
VMSolidus pushed a commit that referenced this pull request Sep 12, 2024
# Description

With #858 (Make
Height Sliders Affect Your Bloodstream Volume) being merged, it
introduced variable blood volumes for differing sizes, whereas before
everyone had the same blood volume (300u).

Because Blood Deficiency subtracted a flat amount from the bloodstream,
it resulted in an unintended consequence where small characters died
very quickly due to blood loss, while large characters could live for
hours. This makes the blood loss amount of Blood Deficiency a
**percentage** of the entity's max blood volume, which means that the
time to become low blood is now the same regardless of blood volume.

# Changelog

:cl: Skubman
- fix: Blood Deficiency now makes you lose a consistent percentage of
blood regardless of your blood volume, which is dictated by size. This
makes the time to low blood and death consistent for every character of
all sizes with this trait.
Remuchi pushed a commit to WWhiteDreamProject/wwdpublic that referenced this pull request Oct 19, 2024
# Description

With Simple-Station/Einstein-Engines#858 (Make
Height Sliders Affect Your Bloodstream Volume) being merged, it
introduced variable blood volumes for differing sizes, whereas before
everyone had the same blood volume (300u).

Because Blood Deficiency subtracted a flat amount from the bloodstream,
it resulted in an unintended consequence where small characters died
very quickly due to blood loss, while large characters could live for
hours. This makes the blood loss amount of Blood Deficiency a
**percentage** of the entity's max blood volume, which means that the
time to become low blood is now the same regardless of blood volume.

# Changelog

:cl: Skubman
- fix: Blood Deficiency now makes you lose a consistent percentage of
blood regardless of your blood volume, which is dictated by size. This
makes the time to low blood and death consistent for every character of
all sizes with this trait.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Changes any cs files Changes: YML Changes any yml files Status: Needs Review Someone please review this
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants