Skip to content

Sensing heat#12191

Open
cardillan wants to merge 2 commits into
Anuken:masterfrom
cardillan:sensing-heat
Open

Sensing heat#12191
cardillan wants to merge 2 commits into
Anuken:masterfrom
cardillan:sensing-heat

Conversation

@cardillan

Copy link
Copy Markdown
Contributor

My attempt at implementing Anuken/Mindustry-Suggestions#6353.

As there isn't a heat field common to all blocks, I've added the ability to sense @heat to blocks that define the field, returning the value of the field as-is. Apparently, different blocks use different ranges for heat, so maybe adding a @heatCapacity or @maxHeat property might be useful, but I didn't want to go there as I don't understand the mechanism all that much. It should still be useful for map authors using world processors as it is.

If your pull request is not translation or serverlist-related, read the list of requirements below and check each box:

  • I have read the contribution guidelines.
  • I have ensured that my code compiles, if applicable.
  • I have ensured that any new features in this PR function correctly in-game, if applicable.

@EggleEgg

EggleEgg commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

how does this interact with nuclear reactor heat? it is primordial it does not reach values >1 to not break serpulo logic

@cardillan

Copy link
Copy Markdown
Contributor Author

how does this interact with nuclear reactor heat? it is primordial it does not reach values >1 to not break serpulo logic

It doesn't change the heat values at all anywhere, I've only added the ability to sense them.

@EggleEgg

EggleEgg commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

I see now, HeatBlock is unchanged (nuclear reactor implements it) so sensing is (or should be) exactly the same. My concern was sensor heat returning values >1, as heatFrac()/heatOutput can reach up to 15. This would ve broken quite a lot of basic serpulo logic

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants