widapro / wledPixel

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

2,7.1 temperature issue #54

Open Grey-Lancaster opened 10 months ago

Grey-Lancaster commented 10 months ago

Sorry, still a strange issue.

It does create a sensor but the sensor is somehow one behind of my settings . So it is set for Fahrenheit now. The max display shows 68, mqtt explore shows 68. but my sensor shows 152

If I set it to Celsius the max show 20, mqtt explorer shows 20, but the sensor shows 68

If I change it back to Fahrenheit the sensor shows 152

Grey-Lancaster commented 10 months ago

I see what it is doing. Since my homeassistant is set to Fahrenheit it converts whatever value it gets to Fahrenheit

Grey-Lancaster commented 10 months ago
if (device_type == "ds18b20") {
  sprintf( topic_config,  "homeassistant/sensor/%s/temperature/config", shortMACaddr.c_str() );
  root["name"]                = "wledPixel-" + shortMACaddr + "_Temperature";
  root["uniq_id"]             = "wledPixelLight" + shortMACaddr + "Temperature";
  root["state_topic"]         = "wledPixel-" + shortMACaddr + "/temperature";
  root["device_class"]        = "temperature";
  root["unit_of_measurement"] = "°F";
} 

no clue why I cannot post code properly anyways, I changed it F and it works fine for me now