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 detected while using #1047 & #1010 & #1000 temp sensors #18559

Closed
setar opened this issue Jul 6, 2020 · 11 comments
Closed

Comments

@setar
Copy link
Contributor

setar commented Jul 6, 2020

Bug Description

Thermal runaway is detected at startup and stops Marlin.

My Configurations

setar_conf_bf.20x.zip

Steps to Reproduce

Download Marlin bugfix 2.x after c43bbcc
define TEMP_SENSOR_0 1047 , 1010 or 1000
Compile and install
Restart
Expected behavior: Normal Marlin startup

Actual behavior: Thermal runaway detection "MAX TEMP on E1" or something like that.

Additional Information

in termistors.h type must be int16_t, but it not all , need additional search and debug

#define PtAdVal(T,R0,Rup) (short)(1024 / (Rup / PtRt(T, R0) + 1))

@sjasonsmith
Copy link
Contributor

@setar did this work for you in older Marlin versions? Assuming so, do you know what the last version was that it worked for you in?

@setar
Copy link
Contributor Author

setar commented Jul 6, 2020

today i work on 70fa4c9 with rebase my hardware conf.

@ellensp
Copy link
Contributor

ellensp commented Jul 7, 2020

Definitely effects

  • 1047 : Pt1000 with 4k7 pullup (E3D)
  • 1010 : Pt1000 with 1k pullup (non standard)
  • 147 : Pt100 with 4k7 pullup
  • 110 : Pt100 with 1k pullup (non standard)
  • 1000 : Custom - Specify parameters in Configuration_adv.h

@ellensp
Copy link
Contributor

ellensp commented Jul 7, 2020

@setar I don't have these thermistor types. but to me it look like they are PTC not NTC
But marlin hasn't been told this.

Can you try something for me?

To tell marlin you have a PTC you add the line #define REVERSE_TEMP_SENSOR_RANGE_NNNN 1 to the thermistor table.

Eg for table 1047
add the line #define REVERSE_TEMP_SENSOR_RANGE_1047 1 to the file Marlin/src/module/thermistor/thermistor_1047.h
after the "#pragma once" line.

Eg for table 147
add the line #define REVERSE_TEMP_SENSOR_RANGE_147 1 to the file Marlin/src/module/thermistor/thermistor_147.h
after the "#pragma once" line.

Does this now read correctly and behave as it should?

@setar
Copy link
Contributor Author

setar commented Jul 7, 2020

change need to be applied to current version bugfix-2.0.x ?
e.g 588d40a
I will check it after 6-7 hours

@ellensp
Copy link
Contributor

ellensp commented Jul 7, 2020

Yes a current bugfix that has the issue

@heppstar
Copy link

heppstar commented Jul 7, 2020

Hi,

I have the same problem. You also get it with "#define TEMP_SENSOR_0 999" that I use for my CNC-machine. Can not start it now.

Regards,
heppstar

@ellensp
Copy link
Contributor

ellensp commented Jul 7, 2020

@heppstar Please don't hijack this issue. Issues relating to thermistor type 999 and 998 have already been reported and fixed in bugfix see #18525

@setar
Copy link
Contributor Author

setar commented Jul 7, 2020

#define REVERSE_TEMP_SENSOR_RANGE_1047 1

the fix work fine:
Normal Marlin startup complete,
and I checked the temperature with an external thermocouple on the multimeter at 25, 50, 100, 150, 250 C - everything is fine, the difference is <1C

ellensp pushed a commit to ellensp/Marlin that referenced this issue Jul 8, 2020
@ellensp
Copy link
Contributor

ellensp commented Jul 8, 2020

This is now fixed and merged into latest bugfix

@ellensp ellensp closed this as completed Jul 8, 2020
@github-actions
Copy link

github-actions bot commented Sep 6, 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 Sep 6, 2020
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

5 participants