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

Heating failed #3085

Closed
mtm88001 opened this issue Mar 5, 2016 · 13 comments
Closed

Heating failed #3085

mtm88001 opened this issue Mar 5, 2016 · 13 comments

Comments

@mtm88001
Copy link

mtm88001 commented Mar 5, 2016

Using repetier host, heat hot end up to working temperature. Start a print then select kill print. Start print again before the extruders cool off below the cold extrusion prevention setting. Heating fails and requires reboot of Ramps board. This behavior does not happen in Release firmware. I have been able to duplicate this behavior on the RC branch as well as the RCBugFix

@jbrazio
Copy link
Contributor

jbrazio commented Mar 6, 2016

Do you have the USE_WATCHDOG flag set ?

@jbrazio
Copy link
Contributor

jbrazio commented Mar 6, 2016

Btw, when you say "heating fails" what appears in the screen or in the serial console before locking up ?

@mtm88001
Copy link
Author

mtm88001 commented Mar 6, 2016

yes I have watch dog set. I get the error message heading failed on lcd,
as soon as I get a chance I'll re run and copy the console output.
On 6 Mar 2016 04:10, "João Brázio" notifications@github.com wrote:

Btw, when you say "heating fails" what appears in the screen or in the
serial console before locking up ?


Reply to this email directly or view it on GitHub
#3085 (comment)
.

@jbrazio
Copy link
Contributor

jbrazio commented Mar 6, 2016

Please have a look at #3077.

@jbrazio
Copy link
Contributor

jbrazio commented Mar 6, 2016

But this could be another bug, when possible I'd test it in my printer. Which Marlin version are you exactly using ?

@mtm88001
Copy link
Author

mtm88001 commented Mar 7, 2016

I have tried this on both the RC3 and RCBugFix
configH.zip

@thinkyhead
Copy link
Member

@mtm88001 I think I might have a solution. In the function disable_all_heaters change these lines:

target_temperature[NR] = 0; \
. . .
target_temperature[0] = 0;

to

setTargetHotend(NR, 0); \
. . .
setTargetHotend(0, 0);

thinkyhead added a commit to thinkyhead/Marlin that referenced this issue Mar 11, 2016
Addressing MarlinFirmware#3085. If the target_temperature is set directly it never invokes or disables the timer set in `start_watching_heater`, so after _stopping a print_ or turning off _all heaters_ you could get a “Heating failed” error.
@mtm88001
Copy link
Author

Thanks, I'll give it a try and report back
On 11 Mar 2016 07:02, "Scott Lahteine" notifications@github.com wrote:

I think I might have a solution. In the function disable_all_heaters
change these lines:

target_temperature[NR] = 0;
. . .
target_temperature[0] = 0;

to

setTargetHotend(NR, 0);
. . .setTargetHotend(0, 0);


Reply to this email directly or view it on GitHub
#3085 (comment)
.

@jbrazio
Copy link
Contributor

jbrazio commented Mar 13, 2016

@mtm88001 Now that #3132 has been merge to RCBugfix. Did you had the opportunity to validated if the issue has been fixed ?

@mtm88001
Copy link
Author

not yet, I'll test Monday as soon as I get a chance. Thanks for looking
into the issue.
On 13 Mar 2016 14:46, "João Brázio" notifications@github.com wrote:

@mtm88001 https://github.com/mtm88001 Now that #3132
#3132 has been merge to
RCBugfix. Did you had the opportunity to validated if the issue has been
fixed ?


Reply to this email directly or view it on GitHub
#3085 (comment)
.

@mtm88001
Copy link
Author

downloading the latest rcbugfix, I was not able to reproduce the heading
failed error. I did a test print and at the end of the print, the z axis
reversed and sent the head into the bed. haven't had a chance to repeat yet.
On 6 Mar 2016 04:08, "João Brázio" notifications@github.com wrote:

Do you have the USE_WATCHDOG flag set ?


Reply to this email directly or view it on GitHub
#3085 (comment)
.

@thinkyhead
Copy link
Member

Feeling pretty confident about this one. And… RC4 is out!

@jbrazio jbrazio modified the milestone: 1.1.0 Jul 16, 2016
@github-actions
Copy link

github-actions bot commented Apr 6, 2022

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 6, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants
@jbrazio @thinkyhead @mtm88001 and others