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

[BUG] Thermal runaway protection error even after changing PIDs, Works fine on stock firmware #15088

Closed
Haxk20 opened this issue Aug 29, 2019 · 12 comments

Comments

@Haxk20
Copy link
Contributor

Haxk20 commented Aug 29, 2019

I have tevo tarantula PRO and the stock firmware works just fine with its PID. Upgraded to newest marlin 2.0 branch and updated all the pids because the stock ones were giving me runaway protection error on bed. But the same happens with the new PIDs and i basically cannot print because when i set it to 50C it reaches 41C and stays there. But weirdly when i set 60C it reaches 51C just fine and then errors out.
Configuration stock.zip
Configuration Marlin 2.0.zip
I atached zips of both stock and marlin 2.0 configs for my printer.
Marlin 2.0 configs already includes tuned PIDs.

@boelle
Copy link
Contributor

boelle commented Aug 29, 2019

could it be that the bed is simply just to slow?

try and use a stop watch and test how fast it reach's 40, 50, 60.... Does it take longer to reach 60 than 40? if so i would vote on the bed is just to weak

@Haxk20
Copy link
Contributor Author

Haxk20 commented Aug 29, 2019

The bed doesnt have this issue on marlin 1 which is stock firmware and has thermal runaway protection enabled too so the bed is OK.
The issue here is that the bed stops always 9C below the target and then throws out error. If i set target to 70C it would get to 61C and after minute or so will error. If i set it to 60C it would reach 51C and so on.

@boelle
Copy link
Contributor

boelle commented Aug 29, 2019

is #define THERMAL_PROTECTION_BED_PERIOD also set to the same values?
and also #define THERMAL_PROTECTION_BED_HYSTERESIS

@thisiskeithb
Copy link
Member

thisiskeithb commented Aug 29, 2019

In your Configuration stock.zip, PIDTEMPBED is disabled (Tevo's Tarantula Pro repo has PIDTEMPBED disabled as well), so it's using bang-bang to heat the bed. I can't open your Configuration Marlin 2.0.zip, but the Tarantula Pro config in bugfix-2.0.x has it enabled with default (untuned) values.

If you comment out PIDTEMPBED, does the bed heat as it should?

Alternatively, you can find your own bed PID values by running a PID tune. With PIDTEMPBED enabled and the bed at room temperature (this is important) run M303 C6 E-1 S60. What are the reported new values?

So we have a couple options:

  1. Update the Tarantula Pro config using your PID values when you run a PID tune on the bed or
  2. Disable PIDTEMPBED on the Tarantula Pro config and let it use bang-bang if that works fine

@Haxk20
Copy link
Contributor Author

Haxk20 commented Aug 29, 2019

Im in the middle of big print using stock firmware so i will try once the print is done.

@Haxk20
Copy link
Contributor Author

Haxk20 commented Aug 29, 2019

@thisiskeithb But as i said i updated the PIDs in the new configuration.h and uploaded. It does the same thing.

@thisiskeithb
Copy link
Member

I'm leaning toward disabling PIDTEMPBED in the config.

@boelle
Copy link
Contributor

boelle commented Aug 29, 2019

same her, just did not think of disabling pid for the bed

what could be the reason for them doing that?

@thisiskeithb
Copy link
Member

I put in a PR (#15089) to disable PIDTEMPBED since that is what stock uses.

Users can always enable it and run a PID tune to get values for their machine if so desired.

@Haxk20
Copy link
Contributor Author

Haxk20 commented Aug 29, 2019

Thanks

@Haxk20
Copy link
Contributor Author

Haxk20 commented Aug 29, 2019

Confirming. Issue is fixed.

@Haxk20 Haxk20 closed this as completed Aug 29, 2019
@github-actions
Copy link

github-actions bot commented Jul 4, 2020

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 Jul 4, 2020
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

3 participants