search

widapro / wledPixel

Dot matrix panel management
MIT License
73 stars 12 forks source link

DS18b20 temp not published #48

Closed Grey-Lancaster closed 1 year ago

Grey-Lancaster commented 1 year ago

I guess this is supposed to do it, but it never shows the temp

      if (mqttEnable) mqttClient.publish(String("wledPixel-" + shortMACaddr + "/temperature").c_str(), dsTemp.c_str());

image

mats-nk commented 1 year ago

That is NOT the internal temperature, its the External Dallas OneWire DS18B20 temp sensor.

See https://github.com/widapro/wledPixel/raw/9ea4164575de9fc7c5ffbd9eb50f75493889256c/img/WledPixel%20Wiring800.png

Grey-Lancaster commented 1 year ago

I did not say it was the internal temp. There is no internal temp? The code says to publish dsTemp.c_str());

Sorry, I guess my title does say internal temp.

Grey-Lancaster commented 1 year ago

not sure what the diff is but other builds have made a lot more entities tan now. Not sure if it is "breaking" at setting the temp

image

\ vs

image

Grey-Lancaster commented 1 year ago

Broken

widapro commented 1 year ago

It was a change in Home Assistant MQTT discovery for sensor. I fixed it in v2.7.1