Skip to content

Conversation

@dynst
Copy link
Contributor

@dynst dynst commented Jul 19, 2025

Summary

Balance "More realistic meat calories"

Purpose of change

Uncooked meat realistically should have excess water weight and bulk removed in the cooking process.

Cooked meat should have much more calories per gram, but also its density is physically impossible right now #54811

And realistically speaking, cracklins shouldn't be almost 2x denser than water.

Describe the solution

Increase the volume of all cracklins, make them 4x less space efficient but with portions still divisible into 500 ml containers #81781

Increase the calories in cooked meat and raw meat and everything that inherits from them.

Fix the density of meat_cooked and the numerous items that copy-from inherit from it.

Increase the weight and volume of raw meat. Update all items that inherit from raw meat accordingly.

Make meat_cooked inherit from meat and use "relative" and "proportional" fields.

Describe alternatives you've considered

Increasing the weight of raw meat, but that involves recipe consistency checks again. #61746 (comment)

Decreasing the volume of cooked meat, but that's impossible because it has to exactly fit into sealed cans. #81781

Testing

Additional context

Raw meat used to also have the illogical density and weight of 296 grams that cooked meat currently still does, until raw meat was fixed in #61746

dynst added 2 commits July 19, 2025 06:29
uncooked meat has excess water weight and bulk.

cooked meat should have much more calories per gram, but also
fewer grams in a 250ml portion.

cracklins definitely don't sink in water.
@github-actions github-actions bot added [JSON] Changes (can be) made in JSON Items: Food / Vitamins Comestibles and drinks Code: Tests Measurement, self-control, statistics, balancing. Game: Balance Balancing of (existing) in-game features. labels Jul 19, 2025
@GuardianDll
Copy link
Member

I would prefer weight be untouched, as a lot of recipes are made based on the old weight, and instead would just reduce the calorie amount

@github-actions github-actions bot added astyled astyled PR, label is assigned by github actions json-styled JSON lint passed, label assigned by github actions labels Jul 19, 2025
@dynst
Copy link
Contributor Author

dynst commented Jul 19, 2025

I would prefer weight be untouched, as a lot of recipes are made based on the old weight, and instead would just reduce the calorie amount

Do you mean reduce the calories in raw meat, rather than make raw meat larger? Then calories would magically appear out of nothing when you cook it into cooked meat.

@GuardianDll
Copy link
Member

GuardianDll commented Jul 19, 2025

Calories would not appear, in most cases, because the game, in most cases again, inherit the amount of calories of a final product from it's components, if item is crafted. There is also a test that ensures that default amount of calories matches the amount of calories that can be obtained via the recipe

But yeah it's probably a choice do you want to fight with recipes using incorrect amount of meat, or fight items not having a correct amount of calories

@dynst dynst marked this pull request as draft July 19, 2025 19:42
@dynst dynst marked this pull request as ready for review July 19, 2025 21:07
dynst added 6 commits July 19, 2025 21:36
it was never explicitly set before.
no more 402 calorie portions.
the 'meat' item is unchanged in mass and will be hard to change.
cooked meat has more calories now so hunger time is longer.
@github-actions github-actions bot added [C++] Changes (can be) made in C++. Previously named `Code` Items: Containers Things that hold other things labels Jul 19, 2025
for a lower density item with more implied surface area.
@dynst dynst changed the title update meat (caloric) density fix meat (caloric) density Jul 19, 2025
@github-actions github-actions bot added the <Bugfix> This is a fix for a bug (or closes open issue) label Jul 20, 2025
@dynst
Copy link
Contributor Author

dynst commented Jul 20, 2025

Simplified to just fixing the number of calories in a portion of meat, can this PR be backported to 0.I? Is there a way to request it from the migo-chan bot?

@GuardianDll
Copy link
Member

Is there a reason to backport it? we limit it only to bugfixes, and meat calories fit it only very nominally in my eyes

@dynst
Copy link
Contributor Author

dynst commented Jul 21, 2025

we limit it only to bugfixes,

Is that actually true? #81649 doesn't say bugfixes only, it doesn't even give absolute rules at all, it's very flexible-sounding guidance.

Balance changes, such as adjusting item weights or spawn chances etc, is more likely to be fine as long as it's well vetted.

And the current test failure on this PR was fixed in #81926

@GuardianDll
Copy link
Member

Need to rebase

@github-actions github-actions bot added json-styled JSON lint passed, label assigned by github actions and removed json-styled JSON lint passed, label assigned by github actions labels Jul 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

astyled astyled PR, label is assigned by github actions <Bugfix> This is a fix for a bug (or closes open issue) [C++] Changes (can be) made in C++. Previously named `Code` Code: Tests Measurement, self-control, statistics, balancing. Game: Balance Balancing of (existing) in-game features. Items: Containers Things that hold other things Items: Food / Vitamins Comestibles and drinks [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants