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

Fix muscle-powered vehicles causing fatigue and thirst #49091

Merged

Conversation

natsirt721
Copy link
Contributor

@natsirt721 natsirt721 commented May 30, 2021

Summary

Bugfixes "Fix muscle-powered vehicles causing fatigue and thirst"

Purpose of change

#47512 removed the fatigue penalty from most fatiguing tasks when weariness was implemented, however a similar change was not applied to muscle-powered vehicles. This PR removes the fatigue and thirst penalties in line with that change.

See #49074 for detailed rationale.

Fixes #46663

Describe the solution

Removed the code responsible for decrementing player fatigue and thirst as a function of load.

Describe alternatives you've considered

Considered leaving thirst in to simulate perspiration. Did not do this consistent with #47512.
Considered adjusting activity level based on load factor. Not comfortable enough to make balance calls like that or dedicated enough to playtest.
Considered that this is intentional, but it seems out-of-line with the rest of the fatigue->weariness changes.

Testing

Playtesting: Created a STR 8 character with no additional features, spawned a brand new bicycle and rode in circles. Took ~2hrs 45min for activity to drop from Fresh to Light. Player was not thirsty or hungry. Total weight was ~75kg. Did not test with a boat.

Additional context

It seems reasonable that the load factor should affect the activity level, but I don't have a solid enough grasp on either the vehicle code or the weariness mechanics to determine what loadings deserve what activity level. This is probably worth further discussion.

@actual-nh actual-nh added <Bugfix> This is a fix for a bug (or closes open issue) Game: Balance Balancing of (existing) in-game features. Mechanics: Character / Player Character / Player mechanics Vehicles Vehicles, parts, mechanics & interactions labels May 30, 2021
@actual-nh actual-nh requested a review from mlangsdorf May 30, 2021 20:00
@actual-nh
Copy link
Contributor

Something that may be creating some confusion is that "fatigue" actually means "need for sleep". Is that your difficulty, @sayke, or is there something else?

@Salty-Panda
Copy link
Contributor

Salty-Panda commented May 30, 2021

Something that may be creating some confusion is that "fatigue" actually means "need for sleep". Is that your difficulty, @sayke, or is there something else?

If you expand the code upward, there is properly handled activity (set to EXERCISE), bionics too.
He only removed the outdated additional cost in thirst and fatigue.

@sayke
Copy link
Contributor

sayke commented May 31, 2021

Got it, sounds good, thank you!

@actual-nh
Copy link
Contributor

The vehicle ramp test on one Travis build (ming) went wrong because the motorcycle ran into a zombie, BTW.

@actual-nh actual-nh requested review from esotericist and removed request for anothersimulacrum June 1, 2021 18:41
@esotericist
Copy link
Contributor

while I want to ask for load awareness at the same time as we drop the old penalties, we don't do that for walking yet either so it's hard for me to insist. future problems

I couldn't find an open issue or anything about getting perspiration effects added to sustained activity, so we probably don't want that forgotten either.

@Salty-Panda
Copy link
Contributor

The perspiration PR does nothing directly with weariness, but if activity levels were to raise body temperature then when/if the PR got implemented it would work with that.

@Rivet-the-Zombie Rivet-the-Zombie merged commit fc3689c into CleverRaven:master Jun 2, 2021
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) Game: Balance Balancing of (existing) in-game features. Mechanics: Character / Player Character / Player mechanics Vehicles Vehicles, parts, mechanics & interactions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Riding a bicycle make you tired instead of weary.
6 participants