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

Add thermal runaway safeguards #17

Closed
hallboyone opened this issue Apr 26, 2023 · 0 comments · Fixed by #18
Closed

Add thermal runaway safeguards #17

hallboyone opened this issue Apr 26, 2023 · 0 comments · Fixed by #18
Labels
enhancement New feature or request safety Critical issues that effect the product's safety

Comments

@hallboyone
Copy link
Owner

There is currently nothing checking that the boiler is responding as expected to the heater inputs. This can lead to thermal runaways should the sensor come detached or fail. Functionality should be added to monitor the state of the boiler and enter an error state should a thermal runaway be sensed.
This could be modeled after the thermal runaway logic in the Marlin 3D printer firmware. The basic rules are

  1. The temp must be changing at some minimum rate when the machine is settling to a temperature (i.e. responding to control inputs)
  2. The temp must not deviate too far from the setpoint after settling (i.e. sensor fails as in CRITICAL - Occasional erratic behavior in boiler #16)
@hallboyone hallboyone added enhancement New feature or request safety Critical issues that effect the product's safety labels Apr 26, 2023
@hallboyone hallboyone linked a pull request May 4, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request safety Critical issues that effect the product's safety
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant