Insteon not initialised in 0.27 release #3056

CBPetrovic · 2016-08-30T16:20:44Z

Home Assistant release (hass --version):
0.27

Python release (python3 --version):
3.4

Component/platform:
Insteon

Description of problem:
Upgraded to 0.27 and now insteon is not getting initialized.

Here's a clip from the log

16-08-29 16:13:24 homeassistant.bootstrap: Error during setup of component insteon_hub
Traceback (most recent call last):
File "/usr/local/lib/python3.4/dist-packages/homeassistant/bootstrap.py", line 157, in setupcomponent
result = component.setup(hass, config)
File "/usr/local/lib/python3.4/dist-packages/homeassistant/components/insteon_hub.py", line 51, in setup
from insteon import Insteon
File "/home/pi/.homeassistant/deps/insteon/init.py", line 3, in
from .insteon import Insteon
File "/home/pi/.homeassistant/deps/insteon/insteon.py", line 15
endpoint=None):
^
SyntaxError: invalid syntax

Expected:

Problem-relevant configuration.yaml entries and steps to reproduce:

Traceback (if applicable):

Additional info:

The text was updated successfully, but these errors were encountered:

Danielhiversen · 2016-08-30T16:30:17Z

@Teagan42 Could you have a look a this one?

Teagan42 · 2016-08-30T16:47:47Z

Well shit - that's an issue with insteon library. We may need to pull out the insteon fan support.

Teagan42 · 2016-08-30T16:48:28Z

I'll see if I can get a fix into the insteon hub library - otherwise we'll revert the insteon changes.

CBPetrovic · 2016-08-30T17:32:35Z

Teagan, Dennis,

I am not a python programmer, but I am familiar with other languages. If
you can help with why that line of code causes the syntax error, I'd be
happy to edit the file (if it's that simple) and see if the insteon hub
initializes. Tell me what to edit, again, if it's simple and localized to
the /deps directory, and I'll try it tonight.

Chris

On Tue, Aug 30, 2016 at 12:48 PM Teagan Glenn notifications@github.com
wrote:

I'll see if I can get a fix into the insteon hub library - otherwise we'll
revert the insteon changes.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#3056 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOdXcvVwnLEQjQCipXGFVkqQ8cgqWXbZks5qlF7mgaJpZM4JwtlP
.

CBPetrovic · 2016-08-30T17:33:41Z

Oh, and I'm sorry Daniel, for calling you Dennis . . . lots of things on my
mind

On Tue, Aug 30, 2016 at 1:32 PM Christopher B. Petrovic <
cbpetrovic@gmail.com> wrote:

Teagan, Dennis,

I am not a python programmer, but I am familiar with other languages. If
you can help with why that line of code causes the syntax error, I'd be
happy to edit the file (if it's that simple) and see if the insteon hub
initializes. Tell me what to edit, again, if it's simple and localized to
the /deps directory, and I'll try it tonight.

Chris

On Tue, Aug 30, 2016 at 12:48 PM Teagan Glenn notifications@github.com
wrote:

I'll see if I can get a fix into the insteon hub library - otherwise
we'll revert the insteon changes.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#3056 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOdXcvVwnLEQjQCipXGFVkqQ8cgqWXbZks5qlF7mgaJpZM4JwtlP
.

Teagan42 · 2016-08-30T17:37:53Z

@CBPetrovic It needs a comma separating the arguments in the method (comma after__version__)

class Insteon(object):
    def __init__(self, username, password, client_id,
                user_agent='insteon_hub/%s' % __version__,
                endpoint=None):

CBPetrovic · 2016-08-30T18:11:11Z

Ok, I'll give that a shot a little later and report back!

Thanks!

Chris

On Tue, Aug 30, 2016 at 1:38 PM Teagan Glenn notifications@github.com
wrote:

@CBPetrovic https://github.com/CBPetrovic It needs a comma separating
the arguments in the method (comma after__version__)

class Insteon(object):
def init(self, username, password, client_id,
user_agent='insteon_hub/%s' % version,
endpoint=None):

—
You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub
#3056 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOdXct1Rk3ikqUULEDV07n7ER5wp8Tliks5qlGp7gaJpZM4JwtlP
.

CBPetrovic · 2016-08-30T18:53:01Z

I tried :-)

CBPetrovic · 2016-08-30T18:53:54Z

Ok, the comma was added where instructed, and now this is the error:

16-08-30 14:43:40 homeassistant.bootstrap: Error during setup of component
insteon_hub
Traceback (most recent call last):
File "/usr/local/lib/python3.4/dist-packages/homeassistant/bootstrap.py",
line 157, in _setup_component
result = component.setup(hass, config)
File
"/usr/local/lib/python3.4/dist-packages/homeassistant/components/insteon_hub.py",
line 51, in setup
from insteon import Insteon
File "/home/pi/.homeassistant/deps/insteon/init.py", line 3, in

from .insteon import Insteon
File "/home/pi/.homeassistant/deps/insteon/insteon.py", line 7, in

from .authorization import InsteonAuthorizer
File "/home/pi/.homeassistant/deps/insteon/authorization.py", line 6, in

from .api import InsteonAPI
File "/home/pi/.homeassistant/deps/insteon/api.py", line 21, in
class InsteonAPI(object):
File "/home/pi/.homeassistant/deps/insteon/api.py", line 94, in InsteonAPI
def unauth_post(cls, path, data, endpoint=CLOUD_API_UL):
NameError: name 'CLOUD_API_UL' is not defined

Found and fixed mispelling (missing "R" ) in file api.py

Which then results in this message:
6-08-30 14:47:46 homeassistant.bootstrap: Error during setup of component
insteon_hub
Traceback (most recent call last):
File "/usr/local/lib/python3.4/dist-packages/homeassistant/bootstrap.py",
line 157, in _setup_component
result = component.setup(hass, config)
File
"/usr/local/lib/python3.4/dist-packages/homeassistant/components/insteon_hub.py",
line 58, in setup
INSTEON = Insteon(username, password, api_key)
File "/home/pi/.homeassistant/deps/insteon/insteon.py", line 19, in
init
self.api = InsteonAPI(self.authorizer, client_id, user_agent, endpoint)
File "/home/pi/.homeassistant/deps/insteon/api.py", line 26, in init
self.endpoint = format_endpoint(endpoint if endpoint else CLOUD_API_URL)
NameError: name 'format_endpoint' is not defined

Now, this is beyond my skillset...

On Tue, Aug 30, 2016 at 2:10 PM Christopher B. Petrovic <
cbpetrovic@gmail.com> wrote:

Ok, I'll give that a shot a little later and report back!

Thanks!

Chris

On Tue, Aug 30, 2016 at 1:38 PM Teagan Glenn notifications@github.com
wrote:

@CBPetrovic https://github.com/CBPetrovic It needs a comma separating
the arguments in the method (comma after__version__)

class Insteon(object):
def init(self, username, password, client_id,
user_agent='insteon_hub/%s' % version,
endpoint=None):

—
You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub
#3056 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOdXct1Rk3ikqUULEDV07n7ER5wp8Tliks5qlGp7gaJpZM4JwtlP
.

Teagan42 · 2016-08-30T19:18:11Z

@CBPetrovic Thanks for the info - I submitted a change to the insteon library back in february that apparently wasn't tested. I'm having him pull out that bit of code and will update when the insteon platform should work again.

CBPetrovic · 2016-08-30T19:38:18Z

No problem. What is interesting is that the 0.26.2 release seemed to work
fine and if I not mistaken, was release not too long ago. Wonder why it
took so long to rear its ugly head?

On Tue, Aug 30, 2016 at 3:18 PM Teagan Glenn notifications@github.com
wrote:

@CBPetrovic https://github.com/CBPetrovic Thanks for the info - I
submitted a change to the insteon library back in february that apparently
wasn't tested. I'm having him pull out that bit of code and will update
when the insteon platform should work again.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#3056 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOdXcpjuThxSLrMd2pTbkTVTLVkN72ILks5qlIH-gaJpZM4JwtlP
.

Teagan42 · 2016-08-30T21:08:54Z

Because I implemented insteon fan support in 27, been sitting on it for
awhile. For now use 0.26.2. Waiting on the owner of the insteon hub library
now.

On Aug 30, 2016 1:38 PM, "CBPetrovic" notifications@github.com wrote:

No problem. What is interesting is that the 0.26.2 release seemed to work
fine and if I not mistaken, was release not too long ago. Wonder why it
took so long to rear its ugly head?

On Tue, Aug 30, 2016 at 3:18 PM Teagan Glenn notifications@github.com
wrote:

@CBPetrovic https://github.com/CBPetrovic Thanks for the info - I
submitted a change to the insteon library back in february that
apparently
wasn't tested. I'm having him pull out that bit of code and will update
when the insteon platform should work again.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<https://github.com/home-assistant/home-assistant/
issues/3056#issuecomment-243549145>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/
AOdXcpjuThxSLrMd2pTbkTVTLVkN72ILks5qlIH-gaJpZM4JwtlP>
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#3056 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC2fZWIhy2DvbFWqH3GQMopLrDwh8trYks5qlIa4gaJpZM4JwtlP
.

Teagan42 mentioned this issue Aug 30, 2016

Bug fix in insteon hub library #3057

Closed

8 tasks

CBPetrovic closed this as completed Aug 30, 2016

CBPetrovic reopened this Aug 30, 2016

Teagan42 mentioned this issue Aug 31, 2016

Reset insteon hub #3062

Merged

8 tasks

robbiet480 closed this as completed in #3062 Sep 2, 2016

Teagan42 mentioned this issue Sep 3, 2016

Insteon Not Working After Upgrade to 27.0/27.1 #3157

Closed

home-assistant locked and limited conversation to collaborators Mar 17, 2017

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

Insteon not initialised in 0.27 release #3056

Insteon not initialised in 0.27 release #3056

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Insteon not initialised in 0.27 release #3056

Insteon not initialised in 0.27 release #3056

Comments

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!