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

Prevent slimespring spawning dehydration #35561

Merged
merged 1 commit into from
Nov 16, 2019

Conversation

anothersimulacrum
Copy link
Member

@anothersimulacrum anothersimulacrum commented Nov 16, 2019

Summary

SUMMARY: Bugfixes "Fix slimespring spawning causing dehydration"

Purpose of change

Fixes #34628
Previously, slimesprings would spawn when you were above your healthy
calories by a certain amount and the amount of water you had after it
was processed through your digestive system would leave you with a
thirst value less than 40 (your 'true' thirst value).

The hunger calculation didn't cause problems, but the thirst calculation
did, because players could spawn lots of slimesprings, which would
massively increase their actual thirst value, pushing them up to dehydrated.
They were not actually dehydrated, because they had the water to recover
in their digestive system, but nonetheless, this was a problem.

Describe the solution

This prevents that by also checking that the player's visible thirst
value is less than a certain amount.
It also makes it so that your 'true' thirst value must be -20 or less.

Describe alternatives you've considered

A different number or not adjusting the necessary 'true' thirst value.

Testing

Created a character, took a lot of slime serum. Debugged pain, hunger, thirst, and fatigue back to 0.
Drank lots of water.
Noticed that I never became dehydrated, unlike before this change.

Previously, slimesprings would spawn when you were above your healthy
calories by a certain amount and the amount of water you had after it
was processed through your digestive system would leave you with a
thirst value less than 40.

The hunger calculation didn't cause problems, but the thirst calculation
did, because players could spawn lots of slimesprings, which would
massively increase their thirst value, pushing them up to dehydrated.
They were not actually dehydrated, because they had the water to recover
in their digestive system, but nonetheless, this was a problem.

This prevents that by also checking that the player's visible thirst
value is less than a certain amount.
It also makes it so that your 'true' thirst value must be -20 or less.
@ZhilkinSerg ZhilkinSerg added <Bugfix> This is a fix for a bug (or closes open issue) [C++] Changes (can be) made in C++. Previously named `Code` Mutations / Traits / Professions/ Hobbies Mutations / Traits / Professions/ Hobbies labels Nov 16, 2019
@ZhilkinSerg ZhilkinSerg merged commit 7796df3 into CleverRaven:master Nov 16, 2019
@anothersimulacrum anothersimulacrum deleted the slime-spawn branch November 16, 2019 22:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Bugfix> This is a fix for a bug (or closes open issue) [C++] Changes (can be) made in C++. Previously named `Code` Mutations / Traits / Professions/ Hobbies Mutations / Traits / Professions/ Hobbies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Omnicellular isn't changing needs properly after spawning slimesprings
2 participants