Detect missing/unresponsive sensor.

[nekitoss]

Hi, @grumpazoid !
Possibly sendHeartbeat(); could help you, but currently domoticz doesn't work correctly with it (without sending data).
I've opened an issue on github, at latest beta gizmocuz make a fix, but as i tested - heartbeat will update last seen time only in devices, not in hardware. - But that if you don't send sensor data, but you are. So try it, but remeber it is beta...
Also check last seen in hardware, not in devices. They may differ.
You may create additional switch, and send opposite bool each time (1-0-1-0-1-0), if you want to do some scripting...
Or just change to a switch type and create dummy alert dvice and update it via script (if switch is 1, then activate alert, and do not deactivate on switch 0)

[grumpazoid]

Thanks for the above comments. This has given me some things to look into, especially the heartbeat command.

I am wondering if MQTT could help. (I have no experience with this yet). From what I have seen MQTT can send out a LWT command if a device is not heard from.

[kimot]

Simply change your log. levels.
If sensor is in idle state - send log.1 to Domoticz - Last Seen will be updated with each log 1 received.
When motion detected - send log.0 - first log.0 received will trigger your action and change Last Seen time too.
If you want detect, if sensor hags in triggered state, you can measure time interval for log 0 by LUA or DzVents script.
If too long - take some action.

By the way - it is problem MySesnsors part in Domoticz.
When I am sending switch status from ESPeasy by JSON call, Last Seen time is updated with each
received value. ON or OFF -both.

[mfalkvidd]

Earlier discussion on the same topic, where we found the root cause: https://forum.mysensors.org/post/29007
Seems like it never got fixed though.

[kimot]

Quick look at Domoticz code
MySensorsBase.cpp

void MySensorsBase::UpdateSwitch(const _eSetType vType, const unsigned char Idx, const int SubUnit, const bool bOn, const double Level, const std::string &defaultname, const int BatLevel)
{
.
.
.
.
.
.

	if (!result.empty())
	{
		if (
			(((vType != V_TRIPPED) || (!bOn))) &&
			((vType != V_SCENE_OFF) && (vType != V_SCENE_ON))
			)
		{
			//check if we have a change, if not do not update it
			int nvalue = atoi(result[0][1].c_str());
			if ((!bOn) && (nvalue == 0))
				return;
			if ((bOn && (nvalue != 0)))
			{
				//Check Level
				int slevel = atoi(result[0][2].c_str());
				if (slevel == level)
					return;
			}
		}
	}

//check if we have a change, if not do not update it

It says all :o(
But interesting is, that for Log 1 value of switch it updates allways.

[grumpazoid]

@kimot Thanks for this. So maybe I could comment out the if statements and then it should always update?

Can I just edit MySensorsBase.cpp and reboot?

[mfalkvidd]

@grumpazoid no you'll need to recompile Domoticz

[grumpazoid]

@mfalkvidd ok thanks....

[grumpazoid]

I have just added in the code to send battery info. Once again, domoticz only updates when the sensor is activated.

Edit: I have managed to get the battery updating as required by creating it's own child id. this now shows as a separate sensor in the Domoticz Utility section. It updates as per the sleep setting.

So for this node I currently have:
Motion ---- last seen = last activation
Battery ---- last seen = every 2 minutes.

[kimot]

Again some Domoticz code - for update battery level - it is blocked by default:

void MySensorsBase::UpdateNodeBatteryLevel(const int nodeID, const int Level)
{
	std::map<int, _tMySensorNode>::iterator ittNode = m_nodes.find(nodeID);
	if (ittNode == m_nodes.end())
		return; //Not found
	_tMySensorNode *pNode = &ittNode->second;
	for (auto & itt : pNode->m_childs)
	{
		itt.hasBattery = true;
		itt.batValue = Level;

		//Uncomment the below to for a sensor update
/*
		for (const auto & itt2 : itt->values)
		{
			if (itt2.second.bValidValue)
			{
				_eSetType vType = itt2.first;
				SendSensor2Domoticz(pNode, &itt, vType);
			}
		}
*/
	}
}

notice - //Uncomment the below to for a sensor update