The Thermosensors plugin does not work

[Andrew Mamohin]

The Thermosensors plugin does not work with the MAX6675 module in version ESP_Easy_mega_20230930 for S2.
The temperature is always equal NAN.
When I flash version 20230822 the plugin works fine.

9BaEAWNEoa.jpg (47.07 KiB) Viewed 4355 times

[Ath]

How is SPI configured?
I assume you've checked the wiring 3 times? (like I usually do if something won't work as expected)

[Andrew Mamohin]

Hi!
Yes, I checked. This device has been working for some time on the firmware 20230822. I tried several times to flash firmware 20230930, but I always get nan.
And now the device works well on 20230822.

Knmp8yM3JZ.jpg (17.44 KiB) Viewed 4333 times

7vxen4EZOP.jpg (10.84 KiB) Viewed 4333 times

[TD-er]

Which exact build are you using? (filename)
I did remove quite a lot of "excessive logs" etc. from that plugin and wrapped it in "#ifndef BUILD_NO_DEBUG" or "#ifndef LIMIT_BUILD_SIZE"
So it might be possible there was something done in such log-only code blocks. But that would probably have been a bug anyway as those would then not be executed if log level was not set.

Have to check the code changes to be sure.

[TD-er]

Ah nope... found the bug I introduced....

https://github.com/letscontrolit/ESPEas ... 3648f18d09

[TD-er]

Can you check this PR when it is ready building?
https://github.com/letscontrolit/ESPEasy/pull/4836

GH Actions build: https://github.com/letscontrolit/ESPEas ... 6389659984

[Andrew Mamohin]

Can you check this PR when it is ready building?

Yes! This build works. Thank you! You fix bugs amazingly quickly!

[TD-er]

Sadly not as quick as I write them...