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

Mysmartblinds don't always react properly #64

Closed
rnewhart opened this issue Mar 2, 2021 · 10 comments
Closed

Mysmartblinds don't always react properly #64

rnewhart opened this issue Mar 2, 2021 · 10 comments
Labels
duplicate This issue or pull request already exists

Comments

@rnewhart
Copy link

rnewhart commented Mar 2, 2021

Bug or issue information
When I ask homekit to close mysmartblinds I get randomly on some shades:

[HOOBS 21EA@Target Position] The write handler for the characteristic 'Target Position' on the accessory 'Office Left' didn't respond at all!. Please check that you properly call the callback!

When I restarted the bridge I get this on all blinds
[HOOBS 21EA@Office Center@@target Position] characteristic was supplied illegal value: number -1 exceeded minimum of 0

I installed
Mysmartblinds Bridge
2.2.4 published 12/2/2020

There is no place to set target position in configuration

Homebridge setup information
HOOBS 3.3.3

Include details about your configuration if needed, but do not include any email address or password.
Include node version, if known.

Logs
[HOOBS 21EA@Office Center@@target Position] characteristic was supplied illegal value: number -1 exceeded minimum of 0
3/2/2021, 3:32:30 PM [HOOBS 21EA@Office Center@@current Position] characteristic was supplied illegal value: number -1 exceeded minimum of 0
3/2/2021, 3:32:30 PM [HOOBS 21EA@Office Center@Office Center Battery@Battery Level] characteristic was supplied illegal value: number -1 exceeded minimum of 0
3/2/2021, 3:32:30 PM [HOOBS 21EA@Living Room Center@@target Position] characteristic was supplied illegal value: number -1 exceeded minimum of 0
3/2/2021, 3:32:30 PM [HOOBS 21EA@Living Room Center@@current Position] characteristic was supplied illegal value: number -1 exceeded minimum of 0
3/2/2021, 3:32:30 PM [HOOBS 21EA@Living Room Center@Living Room Center Battery@Battery Level] characteristic was supplied illegal value: number -1 exceeded minimum of 0
3/2/2021, 3:32:30 PM [HOOBS 21EA@Office Right@@target Position] characteristic was supplied illegal value: number -1 exceeded minimum of 0
3/2/2021, 3:32:30 PM [HOOBS 21EA@Office Right@@current Position] characteristic was supplied illegal value: number -1 exceeded minimum of 0
3/2/2021, 3:32:30 PM [HOOBS 21EA@Office Right@Office Right Battery@Battery Level] characteristic was supplied illegal value: number -1 exceeded minimum of 0
3/2/2021, 3:32:30 PM [HOOBS 21EA@Living Room Left@@target Position] characteristic was supplied illegal value: number -1 exceeded minimum of 0
3/2/2021, 3:32:30 PM [HOOBS 21EA@Living Room Left@@current Position] characteristic was supplied illegal value: number -1 exceeded minimum of 0
3/2/2021, 3:32:30 PM [HOOBS 21EA@Living Room Left@Living Room Left Battery@Battery Level] characteristic was supplied illegal value: number -1 exceeded minimum of 0

@apexad
Copy link
Owner

apexad commented Mar 3, 2021

The API tends to give -1 when the bridge lost communication with the blinds. Try restarting the bridge.

@randymassey
Copy link

randymassey commented Mar 3, 2021 via email

@rnewhart
Copy link
Author

rnewhart commented Mar 3, 2021

I have rebooted the bridge with no help.

3/2/2021, 9:47:33 PM [HOOBS 21EA@Target Position] The write handler for the characteristic 'Target Position' on the accessory 'Living Room Dining' was slow to respond!
3/2/2021, 9:47:33 PM [HOOBS 21EA@Target Position] The write handler for the characteristic 'Target Position' on the accessory 'Office Center' was slow to respond!
3/2/2021, 9:47:33 PM [HOOBS 21EA@Target Position] The write handler for the characteristic 'Target Position' on the accessory 'Living Room Center' was slow to respond!
3/2/2021, 9:47:33 PM [HOOBS 21EA@Target Position] The write handler for the characteristic 'Target Position' on the accessory 'Office Left' was slow to respond!
3/2/2021, 9:47:33 PM [HOOBS 21EA@Target Position] The write handler for the characteristic 'Target Position' on the accessory 'Living Room Right' was slow to respond!
3/2/2021, 9:47:33 PM [HOOBS 21EA@Target Position] The write handler for the characteristic 'Target Position' on the accessory 'Office Right' was slow to respond!
3/2/2021, 9:47:33 PM [HOOBS 21EA@Target Position] The write handler for the characteristic 'Target Position' on the accessory 'Living Room Left' was slow to respond!
3/2/2021, 9:47:36 PM [MySmartBlindsBridge] Office Left currentPosition is now 0
3/2/2021, 9:47:36 PM [MySmartBlindsBridge] Office Right currentPosition is now 0
3/2/2021, 9:47:39 PM [HOOBS 21EA@Target Position] The write handler for the characteristic 'Target Position' on the accessory 'Living Room Dining' didn't respond at all!. Please check that you properly call the callback!
3/2/2021, 9:47:39 PM [HOOBS 21EA@Target Position] The write handler for the characteristic 'Target Position' on the accessory 'Office Center' didn't respond at all!. Please check that you properly call the callback!
3/2/2021, 9:47:39 PM [HOOBS 21EA@Target Position] The write handler for the characteristic 'Target Position' on the accessory 'Living Room Center' didn't respond at all!. Please check that you properly call the callback!
3/2/2021, 9:47:39 PM [HOOBS 21EA@Target Position] The write handler for the characteristic 'Target Position' on the accessory 'Living Room Right' didn't respond at all!. Please check that you properly call the callback!
3/2/2021, 9:47:39 PM [HOOBS 21EA@Target Position] The write handler for the characteristic 'Target Position' on the accessory 'Living Room Left' didn't respond at all!. Please check that you properly call the callback!
3/2/2021, 9:47:45 PM [MySmartBlindsBridge] Office Center currentPosition is now 0
3/2/2021, 9:47:45 PM [MySmartBlindsBridge] Living Room Left currentPosition is now 0
3/2/2021, 9:47:45 PM [MySmartBlindsBridge] Living Room Center currentPosition is now 0
3/2/2021, 9:47:56 PM [HOOBS 21EA@Living Room Dining@Living Room Dining Battery@Battery Level] characteristic was supplied illegal value: number -1 exceeded minimum of 0
3/2/2021, 9:47:56 PM [MySmartBlindsBridge] Living Room Dining currentPosition is now 0
3/2/2021, 9:47:56 PM [HOOBS 21EA@Living Room Right@Living Room Right Battery@Battery Level] characteristic was supplied illegal value: number -1 exceeded minimum of 0
3/2/2021, 9:47:56 PM [MySmartBlindsBridge] Living Room Right currentPosition is now 0

@apexad
Copy link
Owner

apexad commented Mar 3, 2021

I'll look into this soon, been taking a bit of a break from working on homebridge plugins. Hope to get back into it soon.

As far as Node version, there should generally be no issue with using 12.x instead of 14.x but as a note of clarity I am personally running Homebridge Config UI-X, Homebridge 1.3.0 and Node JS 14.x when I tests this plugin.

@randymassey
Copy link

randymassey commented Mar 3, 2021 via email

@apexad
Copy link
Owner

apexad commented Mar 3, 2021

So, I know homebridge 1.3.0 will now throw a lot more into the logs to 'bug' users to contact plugin authors for slow plugins and such.

Seems like HOOBS 3.3.3 is doing the same thing as well, but indeed they will need to verify that. Certainly not a bad thing to be doing this as it makes troubleshooting your accessories a lot easier but there will likely be a need for plugin authors to pay attention to issues more often.

@randymassey
Copy link

randymassey commented Mar 3, 2021 via email

@rnewhart
Copy link
Author

Circling back around to this. I have 7 mysmartblinds and do you think controlling them all at once has anything to do with why some don't respond? The bridge app never fails, but hoobs sending out individual requests to each blind seems to trip it up.

@apexad
Copy link
Owner

apexad commented Mar 17, 2021

Yes, if I could modify this so it could send batch requests that would probably help. I’ll try to get to it soon, but don’t have much time currently.

@apexad apexad added the duplicate This issue or pull request already exists label Apr 5, 2021
@apexad
Copy link
Owner

apexad commented Apr 5, 2021

Closing this in favor of #38 as the fix is the same, need to re-factor this plugin to do batch requests instead of single requests as the bridge is being flooded with requests and sometimes fails.

Hopefully will get to this soon.

@apexad apexad closed this as completed Apr 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

3 participants