Geniushub integration issue

[persil1971]

Home Assistant release with the issue:

0.94.3

Last working Home Assistant release (if known):
N/A

Operating environment (Hass.io/Docker/Windows/etc.):

Component/platform:

/components/geniushub

Description of problem:
Error in log at startup

Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):

Traceback (if applicable):

Fri Jun 14 2019 10:31:19 GMT+0100 (British Summer Time)
_convert_zone(): Failed to convert Timer schedule for Zone 12
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/geniushubclient/__init__.py", line 259, in _convert_zone
    elif setpoint_temp != default_temp:                          # noqa: disable=F821; pylint: disable=used-before-assignment
UnboundLocalError: local variable 'setpoint_temp' referenced before assignment

Additional information:

[ghost]

Hey there @zxdavb, mind taking a look at this issue as its been labeled with a integration (geniushub) you are listed as a codeowner for? Thanks!

This is a automatic comment generated by codeowners-mention to help ensure issues and pull requests are seen by the right people.

[zxdavb]

@persil1971 This is potentially the most embarrassing piece of code I've ever written.

The good new is that I know exactly what is going on, and that there is a fix, which will be made available in the next few days, after some more testing.

[persil1971]

Many thanks for looking into this for me - it's much appreciated. Would you also be able to look at this or should I open a separate issue? Log Details (ERROR) Mon Jun 17 2019 10:10:21 GMT+0100 (British Summer Time) Device 3: Type 'Single Channel Receiver', doesn't match Fingerprint 'Dual Channel Receiver'

…

On Fri, 14 Jun 2019 at 22:00, David Bonnes ***@***.***> wrote: @persil1971 <https://github.com/persil1971> This is potentially the most embarrassing piece of code I've ever written. The good new is that I know exactly what is going on, and that there is a fix, which will be made available in the next few days, after some more testing. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#24530>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AL5WCTW6LR6RHFQFKLA323LP2QBFXANCNFSM4HYGGFGA> .

[zxdavb]

Many thanks for looking into this for me - it's much appreciated. Would you also be able to look at this or should I open a separate issue? Log Details
(ERROR) Mon Jun 17 2019 10:10:21 GMT+0100 (British Summer Time) Device 3: Type 'Single Channel Receiver', doesn't match Fingerprint 'Dual Channel Receiver'

OK, the issue is that I never had a SCR to test against. I will need a copy of your JSON, see:
https://github.com/zxdavb/geniushub-client

... and execute the following:

python ghclient.py ${HUB_ADDRESS} -u ${USERNAME} -p ${PASSWORD} devices -vvv

...and PM me the results at:
https://forum.geniushub.co.uk/t/home-automation-python-cli-client-library-now-available/292/18

[zxdavb]

@persil1971 Could you do the above and, yes, please also open an issue in HA.

[persil1971]

I will do. I need to set up an environment I can install your python library into before I can run the query against my hub.

…

On Tue, 18 Jun 2019 at 21:17, David Bonnes ***@***.***> wrote: @persil1971 <https://github.com/persil1971> Could you do the above and, yes, please also open an issue in HA. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#24530>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AL5WCTSJ4SBIKYKFVB6KXYLP3E7FDANCNFSM4HYGGFGA> .