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

No data published with deepSleep = false? #3

Open
sandeen opened this issue Jan 29, 2019 · 1 comment
Open

No data published with deepSleep = false? #3

sandeen opened this issue Jan 29, 2019 · 1 comment

Comments

@sandeen
Copy link

sandeen commented Jan 29, 2019

My homie-dht is behaving somewhat oddly.

With this config:
"settings":{"type":"sht30","publishInterval":60,"publishForceKey":"temperature","publishForceVal":0.0,"deepSleep":false}

I seem to get no sensor data published, although I do get some heartbeat stuff from the device:

homie/homie-dht4/$stats/signal 72
homie/homie-dht4/$stats/uptime 5819

As a further twist, if I publish a config update:

mosquitto_pub -r -t 'homie/homie-dht4/$implementation/config/set' -f ./homie-dht4

I get temps published again, but if I delete the topic:

mosquitto_pub -r -t homie/homie-dht4/$implementation/config/set -n

the updates stop. (I hope the retained config message doesn't cause constant SPIFFs rewrites of the config?)

@beckenc
Copy link
Owner

beckenc commented Mar 7, 2020

Hi sandeen, sorry for the late response. Do you still have the problem?

I guess the problem is the naming of the HomieSetting forceValSetting, and forceKeySetting in Sensor.hpp.

Could you please change the code of Line 62 and 63 in Sensors.hpp from:
, forceKeySetting("force key", "The key of the ...")
, forceValSetting("force value", "The value for ...)
to
, forceKeySetting("publishForceKey", "The key of the ...")
, forceValSetting("publishForceVal", "The value for ...)

If this fix your problem, please provide a pull request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants