Closed carl689 closed 5 years ago
It seem the issue is that it doesn't find the key 'Revoked'
if device['tags']['Revoked'] == 'false': self.addNode(SenseDetectedDevice(self, self.address, device['id'], device['name']))
Do you have any device detected with Sense ?
Around 28, interestingly the polyglot server lists 28nodes. None in ISY.
I have made some modifications, update to the latest version and give it a try. You may still see the error, but the node should get added.
Still didn't work for me. I ended up deleting the nodeserver and reinstalling.
That got the nodes to add (sorta, some under "Sense" in admin console, others under root isy) but they all always report off a no energy readings. I guess this is to be expected not having an actual API for sense.
I have made a few updates since the Sensable library added a few changes. I still working on one issue about updating the value after initial pull.
I posted an updaded code, you should a least see some value no, but keep in mind still working on it.
Just some quick feedback on the new release (from the Nodelink store). I installed the new version and am now seeing many more Sense devices in my ISY admin console than my Sense app knows about. My Sense app tells me it only knows of 9 devices, but there are 24 showing up in ISY. More than half of them aren't mine, and ones with duplicate names show different values, so it appears they are coming from somewhere other than my Sense device.
Attached is a copy of what's listed in my ISY admin console in case you recognize any of the device names (it's almost as though the data is being pulled for an additional ID that is hardcoded in the app). Nodelink Sense Nodes - 20190621.pdf
Since this is using an unofficial API from sense, I had bug in the previous version that was polling everything from the user sense account, meaning that every account have a lot more devices, many are not completed discovery device or for their own use. I would suggest you delete all the device in the polyglot and then re-run sense node server. This version will only poll the one that are in used.
On Fri, Jun 21, 2019 at 5:48 PM dwengrovitz notifications@github.com wrote:
Just some quick feedback on the new release (from the Nodelink store). I installed the new version and am now seeing many more Sense devices in my ISY admin console than my Sense app knows about. My Sense app tells me it only knows of 9 devices, but there are 24 showing up in ISY. More than half of them aren't mine, and ones with duplicate names show different values, so it appears they are coming from somewhere other than my Sense device.
Attached is a copy of what's listed in my ISY admin console in case you recognize any of the device names (it's almost as though the data is being pulled for an additional ID that is hardcoded in the app). Nodelink Sense Nodes - 20190621.pdf https://github.com/therealmysteryman/udi-sense-monitoring-polyglot/files/3316126/Nodelink.Sense.Nodes.-.20190621.pdf
— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/therealmysteryman/udi-sense-monitoring-polyglot/issues/4?email_source=notifications&email_token=AGZMZRKQ3JZ5TMXACEB2JNTP3VEEFA5CNFSM4HL7GBZKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYJVK7A#issuecomment-504583548, or mute the thread https://github.com/notifications/unsubscribe-auth/AGZMZRNHC3KSRILNAIF4U5DP3VEEFANCNFSM4HL7GBZA .
-- Jean-François j.f.tremblay@gmail.com jf.tremblay@gmail.com
I deleted the Sense nodeserver on my Pi and all of the sense nodes/devices in my ISY, reinstalled from the nodeserver store, and set up my userid and password on Polyglot. The Sense app on my iPhone tells me it recognizes 10 devices in my household. The Sense nodeserver tells me I have 33. I don't recognize any of the names as devices in my home (though it would indeed be nice to have a hot tub, wine fridge and a kegerator which are now showing up in my ISY). There's something definitely wrong with the data being populated.
Sorry about that, I fixed the issue, I was caching the userid/password. You may need to remove and re-add the nodeserver.
JF
On Wed, Jul 3, 2019 at 7:55 PM dwengrovitz notifications@github.com wrote:
I deleted the Sense nodeserver on my Pi and all of the sense nodes/devices in my ISY, reinstalled from the nodeserver store, and set up my userid and password on Polyglot. The Sense app on my iPhone tells me it recognizes 10 devices in my household. The Sense nodeserver tells me I have 33. I don't recognize any of the names as devices in my home (though it would indeed be nice to have a hot tub, wine fridge and a kegerator which are now showing up in my ISY). There's something definitely wrong with the data being populated.
— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/therealmysteryman/udi-sense-monitoring-polyglot/issues/4?email_source=notifications&email_token=AGZMZRNZVBOX4H3UE7QKSTTP5U4AHA5CNFSM4HL7GBZKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZF6SCI#issuecomment-508291337, or mute the thread https://github.com/notifications/unsubscribe-auth/AGZMZRMJVH5F6BMRWCKOKTLP5U4AHANCNFSM4HL7GBZA .
-- Jean-François j.f.tremblay@gmail.com jf.tremblay@gmail.com
Ahhh ... much better, thanks! I am now seeing my devices (and only my devices) in both the ISY and Polyglot. I must say, though, it would have been nice to have that kegerator. :-)
One thing I'm noticing in the log fairly frequently is this: [Controller] [ERROR] query, unable to retrieve Sense Monitor usage: 'Senseable' object has no attribute 'rate_limit'
Is that something I should be concerned with?
haha, that Kegerator is mine ;)
For this one, the author of the library to interact with sense have fixed the issue but didn't push it to pip. The issue you may have is that data doesn't get updated as frequently has is should. Once he push his released that error should disappear.
JF
On Wed, Jul 3, 2019 at 11:22 PM dwengrovitz notifications@github.com wrote:
Ahhh ... much better, thanks! I am now seeing my devices (and only my devices) in both the ISY and Polyglot. I must say, though, it would have been nice to have that kegerator. :-)
One thing I'm noticing in the log fairly frequently is this: [Controller] [ERROR] query, unable to retrieve Sense Monitor usage: 'Senseable' object has no attribute 'rate_limit'
Is that something I should be concerned with?
— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub https://github.com/therealmysteryman/udi-sense-monitoring-polyglot/issues/4?email_source=notifications&email_token=AGZMZRM4IYWUMIEQBCGG27DP5VUGHA5CNFSM4HL7GBZKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZGGNPY#issuecomment-508323519, or mute the thread https://github.com/notifications/unsubscribe-auth/AGZMZRJZJZAXBWI3IRM3PQDP5VUGHANCNFSM4HL7GBZA .
-- Jean-François j.f.tremblay@gmail.com jf.tremblay@gmail.com
I did the install, nodes are visible in polyglot but not coming over to ISY. I see [ERROR] discover: 'Revoked' in the log in polyglot