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

Re-educates marines on safe dietary practices. #17034

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

blackdav123
Copy link
Contributor

@blackdav123 blackdav123 commented Feb 4, 2025

About The Pull Request

Fixes the eating messages using the wrong (10 year old) values.
Marines will be unable to eat if the next bite would make them fat.

Why It's Good For The Game

Better to use our nice pretty nutrition defines instead of incorrect 10 year old numbers.
New players will no longer be fat and slow 24/7.

Changelog

🆑
add: Marines have been better trained about nutrition and will stop eating before they get fat.
fix: Eating messages use the correct nutrition values.
/:cl:

@ilikesaying
Copy link

fat

@Lumipharon
Copy link
Contributor

Imagine fatshaming people out of the game

@TiviPlus TiviPlus added Fix Fixes an issue with the game. Feature New interesting mechanics with new interesting bugs labels Feb 4, 2025
@TheLeopold
Copy link
Contributor

But I want to be fat!

@iforgotmeword
Copy link

iforgotmeword commented Feb 4, 2025

Why not make have a cast time?

"You build up effort to eat more food"

So while you can get fat it just makes trying to

@TheLeopold
Copy link
Contributor

TheLeopold commented Feb 8, 2025

Nanites stealthnerf

@ivanmixo
Copy link
Contributor

ivanmixo commented Feb 9, 2025

Is this not essentially the same thing as removing fatness as a mechanic?

@blackdav123
Copy link
Contributor Author

Is this not essentially the same thing as removing fatness as a mechanic?

yes unless you want to be fat for blood gain or whatever then you can chug nutriment pills

@ivanmixo ivanmixo requested a review from QualityVan February 9, 2025 04:22
Copy link
Contributor

@QualityVan QualityVan left a comment

Choose a reason for hiding this comment

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

Aside from not liking the mechanic, what's the reasoning for the existing fullness thresholds being bugs? To me it looks like they're intentionally wider spread than the defines.

@@ -65,29 +65,29 @@

if(iscarbon(M))
var/mob/living/carbon/C = M
var/fullness = C.nutrition + (C.reagents.get_reagent_amount(/datum/reagent/consumable/nutriment) * 25)
var/fullness = C.nutrition + (C.reagents.get_reagent_amount(/datum/reagent/consumable/nutriment) * 25) + (reagents.get_reagent_amount(/datum/reagent/consumable/nutriment) * 25)
Copy link
Contributor

Choose a reason for hiding this comment

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

What's this formula coming from? It looks like nutriment is 7.5 nutrition / 0.4u, which is significantly less than 50/1. 25/1 is wrong too but it's closer and still overshoots.

Copy link
Contributor

Choose a reason for hiding this comment

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

Oh oops I just glazed entirely over the two reagent holders being separate. Make sure you're only accounting for the nutriment you'll actually get from a bite instead of the entire container.

@blackdav123
Copy link
Contributor Author

blackdav123 commented Feb 9, 2025

Aside from not liking the mechanic, what's the reasoning for the existing fullness thresholds being bugs? To me it looks like they're intentionally wider spread than the defines.

thresholds were all 10 year old apop code and the defines are 5 year old roh code so im assuming there was an oversight when the defines were being put throughout hunger code. there are spots in #3403 where the overfed define is replacing both 450 and 500 so the spots in this pr seem to have just been missed by that one. i dont think its beneficial to have them wider and mislead players about how much nutrition they have.

the equation is something i missed yeah i'll fix it to be the correct amount of nutrition per nutriment, the equation is supposed to be (nutrition in body) + (nutriment nutrition in body) + (nutriment nutrition in the food we're trying to eat)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature New interesting mechanics with new interesting bugs Fix Fixes an issue with the game.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants