Skip to content

Conversation

jwhitak
Copy link
Collaborator

@jwhitak jwhitak commented Aug 29, 2024

They were broken?

yes. yes they were.

What this does

Fixes thermostats so that their controls actually do something now besides make an indefinite clicking noise. The following are a list of the changes:

  • Thermostat controls actually set the temperature now, properly setting and using the correct variable.
    image
    (@brndd's one mistake, uncaught for 5 years)

  • Correctly shows the maximum amounts in C instead of K.
    image

  • Rounds off odd decimal inputs.
    Before:
    image
    After:
    image

  • Allows inputing the exact minimum and maximum, such as the default 0 and 40C.
    Before:
    image
    After:
    This doesn't happen.

  • Allows inputting 0C as a valid input.

  • BONUS: If you try to input a number outside of the range, it'll be clamped to the range. E.g. if you try to drop 99999 in a default thermostat, it'll treat it as 40.

  • Thermostats will now announce if they switch between heating and cooling modes.
    image

Why it's good

I want it warm in my leather making hotroom (thermostats set to 40C, the default maximum, can't get the room warm enough to cure leather quickly...), and I want it cold in the kitchen backroom, like it should be.

How it was tested

The usual - opening up ye olde test server and playing around with all the changes. Heating and cooling arbitarily, trying to set it to 0C to freeze to death, then changing the threshold presets to let me really up the thermostat and cure leather.
Before:
image
After:
image

Changelog

🆑

  • bugfix: Thermostats now properly work when setting new temperatures.

@jwhitak jwhitak added Bug / Fix This is either a bug or a fix for a bug. Grammar / Formatting We is very good at write sentence. 100% tested I promise I tested it every possible way / HAHA HE PROMISED HE TESTED IT AND DIDN'T labels Aug 29, 2024
@SECBATON-GRIFFON
Copy link
Contributor

who touched the thermostat

@toomykins
Copy link
Contributor

amazingly good fix.
hope this also miraculously fixes #35711

Copy link
Collaborator

@west3436 west3436 left a comment

Choose a reason for hiding this comment

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

What is the default control range (max-min) for these? Should be wide enough to prevent constant triggering.

@jwhitak
Copy link
Collaborator Author

jwhitak commented Sep 4, 2024

The control range is that the air alarm's heating/cooling will turn on if it exceeds 2C of the desired temperature, and turn off if it's within 0.5C of the desired temperature. The amount heated/cooled scales based on the difference in desired/actual temperature, and the heat capacity of the gas sample taken.

@west3436 west3436 merged commit e3f2579 into vgstation-coders:Bleeding-Edge Sep 4, 2024
5 checks passed
@jwhitak jwhitak deleted the diltThermostatFix branch September 4, 2024 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

100% tested I promise I tested it every possible way / HAHA HE PROMISED HE TESTED IT AND DIDN'T Bug / Fix This is either a bug or a fix for a bug. Grammar / Formatting We is very good at write sentence.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants