Skip to content

Adjust Hunger rates#1302

Merged
deltanedas merged 7 commits intoDeltaV-Station:masterfrom
Colin-Tel:test
Jun 12, 2024
Merged

Adjust Hunger rates#1302
deltanedas merged 7 commits intoDeltaV-Station:masterfrom
Colin-Tel:test

Conversation

@Colin-Tel
Copy link
Copy Markdown
Contributor

@Colin-Tel Colin-Tel commented Jun 4, 2024

About the PR

Raised hunger rates for all species. Diona had their thirst rate raised to 1.5x the human thirst rate and their hunger is 0.4x the human hunger rate.

Why / Balance

The current hunger rate has relegated chef and botany to not really being consequential in the rounds. Raising the hunger rate can aid against this problem if done carefully.

Technical details

Adjusts the baseDecayRate in HungerComponent.

In a nutshell, hunger is reduced by .05 every second instead of by .001666666.

Changelog

🆑

  • tweak: The rate at which non-Diona species become hungry has slightly increased, while Diona get thirsty more often.

@github-actions github-actions bot added the Changes: YML Changes any yml files label Jun 4, 2024
@deltanedas
Copy link
Copy Markdown
Contributor

vending machines should be more barren, both giving logi a reason to buy restocks and chef more than 0 customers

@Colin-Tel
Copy link
Copy Markdown
Contributor Author

vending machines should be more barren, both giving logi a reason to buy restocks and chef more than 0 customers

Wouldn't it be cool if food&drink vending machines had inventory variation on round start so some will be as full as can be and some could be almost out?

@deltanedas
Copy link
Copy Markdown
Contributor

they can its just usually 1 type is empty thats it, vending machines are still the go to

@Colin-Tel Colin-Tel marked this pull request as ready for review June 5, 2024 00:35
Copy link
Copy Markdown
Contributor

@Axionyxx Axionyxx left a comment

Choose a reason for hiding this comment

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

Instead of changing all these values on the prototype, future proof it by changing the value within HungerComponent.

@Bonktrauma
Copy link
Copy Markdown
Contributor

Resident diona main approves

@github-actions github-actions bot added the Changes: C# Changes any cs files label Jun 5, 2024
@Colin-Tel Colin-Tel requested a review from Axionyxx June 5, 2024 16:45
@github-actions github-actions bot added the S: Needs Review Awaiting review from a Maintainer label Jun 5, 2024
@Colin-Tel
Copy link
Copy Markdown
Contributor Author

This one should be ready.

adjusted diona hunger

Signed-off-by: Colin-Tel <113523727+Colin-Tel@users.noreply.github.com>
@deltanedas deltanedas enabled auto-merge (squash) June 10, 2024 17:36
@deltanedas
Copy link
Copy Markdown
Contributor

should probably be fine on master but if it fails fix :trollface:

@Axionyxx Axionyxx dismissed their stale review June 12, 2024 16:19

If tests pass, I don't care

@deltanedas deltanedas merged commit e6bafb2 into DeltaV-Station:master Jun 12, 2024
@Colin-Tel Colin-Tel deleted the test branch June 14, 2024 20:35
LaryNevesPR pushed a commit to LaryNevesPR/Estacao-Andromeda that referenced this pull request Jan 3, 2025
…tion#1302)

<!--
This is a semi-strict format, you can add/remove sections as needed but
the order/format should be kept the same
Remove these comments before submitting
-->

# Description

<!--
Explain this PR in as much detail as applicable

Some example prompts to consider:
How might this affect the game? The codebase?
What might be some alternatives to this?
How/Who does this benefit/hurt [the game/codebase]?
-->

Adds localization for the Senior Engineer and Senior Physician roles
that were recently enabled.

---

# TODO

<!--
A list of everything you have to do before this PR is "complete"
You probably won't have to complete everything before merging but it's
good to leave future references
-->

n/a

---

<!--
This is default collapsed, readers click to expand it and see all your
media
The PR media section can get very large at times, so this is a good way
to keep it clean
The title is written using HTML tags
The title must be within the <summary> tags or you won't see it
-->

<details><summary><h1>Media</h1></summary>
<p>


![Snímka obrazovky 2024-12-01
164702](https://github.com/user-attachments/assets/ca69e08d-5adb-4e71-aeda-5969d595e429)
![Snímka obrazovky 2024-12-01
165626](https://github.com/user-attachments/assets/18289899-2314-4d21-9b09-9e3e3415a6e0)


</p>
</details>

---

# Changelog

<!--
You can add an author after the `:cl:` to change the name that appears
in the changelog (ex: `:cl: Death`)
Leaving it blank will default to your GitHub display name
This includes all available types for the changelog
-->

:cl: zelezniciar
- fix: Fixed Senior Engineer and Physician names not appearing correctly
LaryNevesPR pushed a commit to LaryNevesPR/Estacao-Andromeda that referenced this pull request Jan 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Changes: C# Changes any cs files Changes: YML Changes any yml files S: Needs Review Awaiting review from a Maintainer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants