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

Heat pump rebalance (TESTMERGE) #6164

Merged
merged 6 commits into from
Dec 8, 2023
Merged

Heat pump rebalance (TESTMERGE) #6164

merged 6 commits into from
Dec 8, 2023

Conversation

Putnam3145
Copy link
Contributor

About The Pull Request

For some reason I can't build on my system anymore so I can't test this locally or else I would. It's annoying.

This changes two things about heat pumps:

  1. As their efficiency decreases, they don't only heat partway to their goal--they'll use more power to reach their goal, up to their standard cap. This leads to identical behavior if you're trying to superheat with them (i.e. set their temperature to either cap) but much "nicer" behavior if both sides are close in temperature. This is a buff to their actual usability but a nerf to their power usage.
  2. Heat pumps now try to heat the entire target network to the given temperature instead of just the output air of the heat pump itself, much like passive gates. This should make them much more capable of actually reaching the target temperature, which they had a hard time doing.

Why It's Good For The Game

Heat pumps aren't really strong enough to replace weird thermal regulator setups even with this, this just makes them significantly more convenient to actually use. They still have the same max power usage and the same max heat pump power in inoptimal temperature delta situations, they're just smarter about using it.

Changelog

🆑
balance: Heat pumps now try harder to heat the output, even in inoptimal conditions (but are otherwise not actually stronger)
/:cl:

@github-actions github-actions bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Nov 24, 2023
silicons pushed a commit that referenced this pull request Nov 29, 2023
silicons pushed a commit that referenced this pull request Nov 30, 2023
silicons pushed a commit that referenced this pull request Dec 6, 2023
silicons pushed a commit that referenced this pull request Dec 8, 2023
silicons pushed a commit that referenced this pull request Dec 8, 2023
@TheLordME TheLordME merged commit 956d41e into master Dec 8, 2023
7 checks passed
@TheLordME TheLordME deleted the heat-pump-rebalance branch December 8, 2023 10:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants