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

[FR] Redundant bed temperature sensors #9162

Closed
echo465 opened this issue Jan 13, 2018 · 5 comments
Closed

[FR] Redundant bed temperature sensors #9162

echo465 opened this issue Jan 13, 2018 · 5 comments
Labels

Comments

@echo465
Copy link

echo465 commented Jan 13, 2018

Currently temperature sensor types are defined in configuration.h like so:

#define TEMP_SENSOR_0 1
#define TEMP_SENSOR_1 0
#define TEMP_SENSOR_2 0
#define TEMP_SENSOR_3 0
#define TEMP_SENSOR_4 0
#define TEMP_SENSOR_BED 1

TEMP_SENSOR_1 can be defined as redundant to TEMP_SENSOR_0 for safety:

#define TEMP_SENSOR_1_AS_REDUNDANT
#define MAX_REDUNDANT_TEMP_SENSOR_DIFF 10

Unfortunately, there doesn't seem to be a way to define a temperature sensor to be redundant to the bed temperature sensor or any other sensor, it can only be configured that TEMP_SENSOR_1 is redundant to TEMP_SENSOR_0. In the interest of increased safety, there should be some way to define a sensor to be redundant to TEMP_SENSOR_BED. More broadly, it would be 'nice to have' for one sensor to be redundant for any other sensor.

My motivation - one of my printers had TEMP_SENSOR_BED vibrate loose and land in the insulation underneath the bed while printing with a bed temp at 60c. The printer responded by increasing power to the heater and the heater was able to keep the sensor reading 60c, so "Thermal Runaway" wasn't triggered. Eventually, I smelled burning and cut power. I measured bed temperature at 125c with an infrared thermometer several minutes after cutting power.

@Spawn32
Copy link
Contributor

Spawn32 commented Jan 13, 2018

Too bad there isn't a up-vote button here :)

Having a redundant sensor to the bed would be nice, even better if we could you use all temp sensors as redundant :)

@thinkyhead thinkyhead added the T: Feature Request Features requested by users. label Feb 11, 2018
@panecki
Copy link

panecki commented May 19, 2020

This should be one of the basic functions. Safety first 👍

@slowbro
Copy link
Member

slowbro commented Jun 4, 2021

I'm trying to give this a go, as it's something I was wanting recently, as well.

@thisiskeithb
Copy link
Member

#22085 has been merged, closing.

Thanks for implementing this feature @slowbro!

@github-actions
Copy link

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked and limited conversation to collaborators Aug 11, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

6 participants