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.

Sign up for GitHub

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

Jump to bottom

More thermal protection #2041

Merged
merged 6 commits into from
May 11, 2015
Merged

More thermal protection #2041

merged 6 commits into from
May 11, 2015

Conversation

thinkyhead
Copy link
Member

  • Enable WATCH_TEMP_PERIOD by default
  • Make the default check for 4°C in 16s so it will catch problems sooner
  • Better function documentation
  • Use next_ms instead of start_ms as a convention
  • If this test fails Stop() the machine just like thermal runaway

@thinkyhead thinkyhead added PR: Improvement Needs: Testing Testing is needed for this change labels May 9, 2015
@thinkyhead thinkyhead mentioned this pull request May 9, 2015
Closed
@thinkyhead thinkyhead removed the Needs: Testing Testing is needed for this change label May 11, 2015
thinkyhead added a commit that referenced this pull request May 11, 2015
@thinkyhead
Merge pull request #2041 from thinkyhead/watch_temp
4097207 
More thermal protection
@thinkyhead thinkyhead merged commit 4097207 into MarlinFirmware:Development May 11, 2015
@thinkyhead thinkyhead deleted the watch_temp branch May 13, 2015 07:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
PR: Improvement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@thinkyhead