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

Hotend Idle Timeout fixes #18032

Conversation

thisiskeithb
Copy link
Member

Requirements

HOTEND_IDLE_TIMEOUT

Description

Use MIN_TO_MS instead of SEC_TO_MS and rename "duration" variable.

Benefits

Instead of X seconds, duration will be X minutes.

Related Issues

None, but see comments from 213d4b8.

Use MIN_TO_MS instead of SEC_TO_MS and rename "duration" variable.
@thisiskeithb thisiskeithb changed the title Hotend Idle Timeout Fixes Hotend Idle Timeout fixes May 18, 2020
@thinkyhead thinkyhead merged commit 1e32df4 into MarlinFirmware:bugfix-2.0.x May 19, 2020
@thisiskeithb thisiskeithb deleted the pr/hotend_idle_timeout_fixes branch May 19, 2020 11:36
@thisiskeithb
Copy link
Member Author

I'm fine with the changes, but why is it preferred to use SEC_TO_MS instead of MIN_TO_MS?

vgadreau pushed a commit to vgadreau/Marlin that referenced this pull request May 29, 2020
Co-authored-by: Scott Lahteine <thinkyhead@users.noreply.github.com>
jmp0x0000 pushed a commit to jmp0x0000/Marlin that referenced this pull request Aug 7, 2020
Co-authored-by: Scott Lahteine <thinkyhead@users.noreply.github.com>
HairingX pushed a commit to HairingX/Marlin that referenced this pull request Jun 16, 2021
Co-authored-by: Scott Lahteine <thinkyhead@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants