manual labour doesn't make you sleepy

[s7eamy]

Summary

Bugfixes "removed mod_fatigue(), mod_stored_nutr(), mod_thirst() calls from various activities"

Purpose of change

Fixes #47391

Describe the solution

by removing the functions calls, manual labour doesn't make you sleepy anymore

[lgtm-com]

This pull request introduces 1 alert when merging f1b6443 into 1e40b42 - view on LGTM.com

new alerts:

• 1 for Empty branch of conditional

[s7eamy]

what should I do about this test failing?

../tests/field_test.cpp:357: FAILED:
(~[slow] ~[.])=>	  CHECK( fields_test_duration() < time_limit )
(~[slow] ~[.])=>	with expansion:
Error: (~[slow] ~[.])=>	  {?} < {?}
(~[slow] ~[.])=>	with message:
(~[slow] ~[.])=>	  Terrain should be irradiated under 5 minutes

[actual-nh]

what should I do about this test failing?

Sigh... it isn't your doing; some of the fields tests have some randomness involved (whether something spreads to another tile, in this case), so they fail (probably more often than they should, but that's a matter for another PR - check out Aivean's comments in #46256 for a bit more background and #47218 for a couple thoughts on this).

[kevingranade]

Looks good to me with or without the trog adjustment, just need a quick test to verify that these activities are still suitably exhausting after the adjustment.

[actual-nh]

Looks like "CR" can be removed. @kevingranade: What is your definition of "adequately fatiguing"? I can get some exact data by running weary-type tests on the various activities in question (looking at things like stamina, fatigue, and morale as well as weariness via debug_weary_info).

[s7eamy]

hey, this actually merged! thanks a lot for your help @actual-nh, i appreciate it a lot!!