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

Enable Thermal Runaway Protection by default #1932

Closed
harerama opened this issue Apr 17, 2015 · 7 comments
Closed

Enable Thermal Runaway Protection by default #1932

harerama opened this issue Apr 17, 2015 · 7 comments

Comments

@harerama
Copy link

It saved my printer once (many thanks !):
I really think security features should be enabled by default and the default configuration seems ok to me.

@harerama harerama reopened this Apr 17, 2015
@thinkyhead thinkyhead added T: Feature Request Features requested by users. Needs: More Data We need more data in order to proceed labels Apr 18, 2015
@thinkyhead
Copy link
Member

I think it was enabled by default at some point, and I forget why it was disabled. But it makes sense to enable it by default, as a safety feature.

@AnHardt
Copy link
Member

AnHardt commented Apr 28, 2015

@thinkyhead
The history tells me. @alexborro added 'Thermal Runaway Protection' on Jun 30 2014 with commented out #defines. This is untouched until now.
Should be save to be enabled by default.

@thinkyhead
Copy link
Member

#1991 enables it for extruders. Do you think it ought to be enabled for the bed as well?

@AnHardt
Copy link
Member

AnHardt commented Apr 29, 2015

Yes, i think so. A heater like the others.

@harerama
Copy link
Author

I think all should be enabled yes.
It saved my printer because I had a cable connection ON THE BED partially cut, so the few remaining wires became really hot (with smoke going out !!!) - as the bed could not maintain its temperature, the protection activated.
As a rule, all safety protections should be enabled by default - letting power users the ability to disable them if they want. We are discussing potentially burning homes here, so the safer the better...

Thanks again for saving mine ;-)

@thinkyhead
Copy link
Member

Done!

@thinkyhead thinkyhead removed the Needs: More Data We need more data in order to proceed label Apr 30, 2015
@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 Apr 21, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants