Presence detection with location accuracy

rkrisi · April 21, 2018, 9:13pm

Dear Community!

I have been using this rule for detecting phones state:

rule "iPhone Home"
when
	Item KristofIPhone_Location changed or
	Item EdinaIPhone_Location changed or
	Item ReKaIPhone_Location changed
then
	val PointType home_location = new PointType(new DecimalType(47.5274), new DecimalType(19.1126))
	val PointType phone_location = triggeringItem.state as PointType
	val nameParts = triggeringItem.name.toString.split("_")
	val triggeringiPhoneName = nameParts.get(0) + "_Home"
	val int distance = phone_location.distanceFrom(home_location).intValue()
	if(distance < 150) {
		postUpdate(triggeringiPhoneName, "ON")
		logInfo(nameParts.get(0), "iPhone is at home.")
	} else {
		postUpdate(triggeringiPhoneName, "OFF")
		logInfo(nameParts.get(0),  "iPhone is away.")
	}
end

However it is not working well at all times. Sometimes phones loses precise location thus the rule says that the phone is not at home. (Because the precision is bigger than that constant number) I want to solve this something like including the Location Accuracy to make a dynamic radius for “Home”. Like:

	val DecimalType locationAccuracy = nameParts.get(0) + "_LocationAccuracy"

and in the head of the if:

if(distance < 150 + locationAccuracy)

But I get some errors like this:

	2018-04-21 11:47:07.416 [ERROR] [ntime.internal.engine.RuleEngineImpl] - Rule 'iPhone Home': An error occurred during the script execution: Could not invoke method: org.eclipse.smarthome.model.script.lib.NumberExtensions.operator_plus(java.lang.Number,java.lang.Number) on instance: null

What I’m missing?

Thanks

opus · April 21, 2018, 9:19pm

I guess your locationAccuracy hasen’t been assigned a value.

rkrisi · April 21, 2018, 9:20pm

One of my LocationAccuracy item:

Number KristofIPhone_LocationAccuracy "Location Accuracy [%.1f m]" <pin> { channel="icloud:device:e4648d7f:bd7e5916:locationAccuracy" }

And I can see the Accuracy on my sitemap…

vzorglub · April 21, 2018, 9:45pm

This is wrong, a DecimalType cannot be the sum of a number and String

val DecimalType locationAccuracy = nameParts.get(0) + "_LocationAccuracy"

Put your location accuracy items in a group gAccuracy

Then do:

val locationAccuracyString = nameParts.get(0) + "_LocationAccuracy"
val locationAccuracy = gAccuracy.members.filter[ i | i.name == locationAccuracyString ].state as DecimalType
if(distance < 150 + locationAccuracy)

rkrisi · April 21, 2018, 10:01pm

Thanks I’ll try that!

rkrisi · April 22, 2018, 11:38am

I get some errors like this:

2018-04-22 13:34:25.444 [ERROR] [ntime.internal.engine.RuleEngineImpl] - Rule 'iPhone Home': 'state' is not a member of 'com.google.common.collect.Iterables$6'; line 12, column 25, length 70

vzorglub · April 22, 2018, 5:09pm

I have never done this so lets ytry together:

val locationAccuracyString = nameParts.get(0) + "_LocationAccuracy"
val locationAccuracyItem = gAccuracy.members.filter[ i | i.name == locationAccuracyString ].head
val locationAccuracy = locationAccuracyItem.state as DecimalType
if(distance < 150 + locationAccuracy)

rkrisi · April 22, 2018, 5:53pm

Thanks this seems to solve the issue I haven’t found a better solution for this on the forum so it can be also good for anyone else. All phones sometimes loses location so comparing it with a constant number is not the best idea…
This is much better!

Thanks again!

vzorglub · April 22, 2018, 6:04pm

That’s great, I wasn’t sure myself, I am going to consolidate my rules with grouping soon so this was good practice.

mjcumming · September 3, 2018, 3:59pm

Below is how I ended up doing this… I am using the LastUpdate event to trigger the rule because Location gets updated before LocationAccuracy which can affect calculations of distance … I haven’t included the items or groups but I think the code is fairly readable without.

rule "Distance from home"

when
	Member of giDeviceLastUpdate received update
then
	Home_Location.state = new PointType(new DecimalType(44.925994), new DecimalType(-93.310812))
	val nameParts = triggeringItem.name.toString.split("_")
	val iDeviceName = nameParts.get(0) 

	val locationAccuracyName = nameParts.get(0) + "_LocationAccuracy"
	val locationAccuracyItem = giDeviceLocationAccuracy.members.findFirst[ i | i.name == locationAccuracyName ]
	val locationAccuracy = locationAccuracyItem.state as DecimalType

	val locationName = iDeviceName + "_Location"
	val locationItem = giDeviceLocation.members.findFirst[ i | i.name == locationName ]

	val int distance = Home_Location.distanceFrom(locationItem)

	val presenceiCloudName = iDeviceName + "_iCloud"
	val iCloudItem = gPresent.members.findFirst[ i | i.name == presenceiCloudName ]

	if (distance < 200 + locationAccuracy) {
		iCloudItem.postUpdate (ON)
 		logInfo("iCloud Testing : ","Present  "+iDeviceName)
	}
	else {
		iCloudItem.postUpdate (OFF)
		logInfo("iCloud Testing : ","Not present  "+iDeviceName)
	}
end

rkrisi · September 3, 2018, 4:03pm

Thanks, that’s good to know. I haven’t had time to dig into it, but I can clearly see that it takes some time to realize that I’m home…
I will try your method, thanks!

mjcumming · September 18, 2018, 2:36am

after experimenting over time, I have changed the rule such that if locationAccuracy < 50 I ignore the update and do not change presence.

rkrisi · September 18, 2018, 10:02am

I’m thinking about the opposite way. If the LocationAccuracy is big, it shouldn’t update the Home or Away. Because sometimes a 6km Accuracy will end in updating the switch as ‘Home’.

Why it shouldn’t update if the location accuracy is smaller than 50m? I usually get around 20m accuracy, which means it will never update the switch

mjcumming · September 18, 2018, 12:54pm

Sorry, meant > rather than <… agree, only update when location is accurate

mjcumming · September 30, 2018, 12:14am

Is there a way to use the location that is specified during configuration using the paperui rather than hard coding in the rules?

rkrisi · September 30, 2018, 7:20pm

You mean the location which is specified in the PaperUI Regional settings?

mjcumming · September 30, 2018, 8:39pm

Yes, that one!

rbergen · October 22, 2018, 4:40am

Would you be able to share your items file? I tried copying / pasting your code in my rules but am getting errors:

2018-10-21 22:34:41.067 [ERROR] [ntime.internal.engine.RuleEngineImpl] - Rule 'iPhone Home': An error occurred during the script execution: Couldn't invoke 'assignValueTo' for feature JvmVoid:  (eProxyURI: icloud.rules#|::0.2.0.2.0.0::0::/2)

I have groups created with the devices in there.

Group gPresent
Group giDeviceLocationAccuracy
Group giDeviceLocation

//Rich iPhone
Switch RichPhone_iCloud 			"Rich is at home"		(gPresent)
Number RichPhone_LocationAccuracy 	"Rich Location Accuracy"  	(giDeviceLocationAccuracy)	{ channel="icloud:device:3c1b5a15:4a5be5fe:locationAccuracy" }
Location RichPhone_Location 		"Rich Location"  		(giDeviceLocation)	{ channel="icloud:device:3c1b5a15:4a5be5fe:location" }

Not sure what I’m missing or doing wrong at this point.

mjcumming · October 22, 2018, 3:04pm

Richard, I just in the middle of revising my items and rules around presence and the icloud. Will be a bit before I can post anything useful…

Normann_P_Nielsen · February 11, 2019, 7:48pm

Old post, but could You share the Presence stuff?