[BUG] Thermal runaway detected while using #999 & #998 virtual temp sensors

[Patag]

Bug Description

Thermal runaway is detected at startup and stops Marlin.

My Configurations

BTT SKR Pro v1.1 on test bench
Conf.zip

Steps to Reproduce

1. Download Marlin bugfix 2.x after 30/06 (don't know exactly what commit)
2. define TEMP_SENSOR_0 999 and TEMP_SENSOR_BED 998
3. Compile and install
4. Restart

Expected behavior: Normal Marlin startup

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

Additional Information

This happend after 30/06 commit

[ellensp]

If you give any heating gcodes you can still trigger a Thermal runaway.

[Patag]

No gcodes are sent at all

[ellensp]

Confirmed on a standard ramps current bugfix. Only config changes was to enable a REPRAP_DISCOUNT_FULL_GRAPHIC_SMART_CONTROLLER and set TEMP_SENSOR_0 999

[ellensp]

Seem to have been introduced int this commit. c43bbcc

[ellensp]

See the PR for how to get around this issue.

[Patag]

Works great. Thank you. Do you want me to close this issue or do we wait for official merge ?

[sjasonsmith]

@Patag I suggest you wait for the PR to merge.

[sjasonsmith]

Ideally you would re-test after the merge to be sure it is fixed. Sometimes PRs change during the PR process and are merged before bugs are found.

[Patag]

OK

[setar]

8a594fc - Not fixed all problem:
I have the same bug, MSG_ERR_MAXTEMP on E1
But i not use 999 and 998 sensors .

#define TEMP_SENSOR_0 1047
#define TEMP_SENSOR_1 0
--skip--
#define TEMP_SENSOR_BED 1
#define TEMP_SENSOR_PROBE 0
#define TEMP_SENSOR_CHAMBER 13

#define TEMP_BED_PIN P0_23_A0 // A0 (TH1)
#define TEMP_0_PIN P0_24_A1 // A1 (TH2)
#define TEMP_CHAMBER_PIN P0_25_A2 // A2 (TH3)

setar_conf_bf.20x.zip

UPDATE:
Sorry the different problem, but similar.
reproduced with 1047 , 1010 and other sensor, where used "PtLine" temp define.

UPDATE2:
reproduced also with 1000 (Custom Thermistor)

#define TEMP_SENSOR_0 1000

#if TEMP_SENSOR_0 == 1000
#define HOTEND0_PULLUP_RESISTOR_OHMS 4700 // Pullup resistor
#define HOTEND0_RESISTANCE_25C_OHMS 1097 // Resistance at 25C // pt1000 = 1097,40 Ohm
#define HOTEND0_BETA 3850 // Beta value //pt1000 = 3850
#endif

[sjasonsmith]

@setar, please do not add your unrelated issue to this one. This issue is specific to simulated sensors 998 and 999.

@Patag, this has now been merged. There were some changes made prior to merging, so you may want to re-test it before closing the issue.

[ellensp]

I have just retested with merged changes. They work fine.
So i'm closing this.
@setar If your unrelated issues still persists then please open a new issue.

[github-actions]

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.