Merging insteon_plm and insteonlocal

The underlying library for insteonlocal is no longer being developed. I have been working with some of those developers to bring hub support to the insteon_plm component. It is close to ready but based on that “merger” it would make sense to do one of two things:

  1. Decommission insteonlocal component and make insteon_plm the only Insteon component
  2. Decommission insteonlocal and rename insteon_plm to insteon.

Option 1 is easier and less change but option 2 may be better for the overall visibility of insteon as a component. What are other people’s thoughts? Keep in mind there is an old insteon component that was the predecessor to the insteonlocal. It was decommissioned a while ago due to issues with the approach they took.

When this is done, should the hub have the same device support as a PLM? I have a motion sensor that appears to not work with the current insteonlocal.

Yes, it will. Currently the motion sensor is supported including the light sensor and the battery sensor.

1 Like

@balloob sorry to push this at you but I would really value your input here. I was hoping to get more input from the community but I need to make a decision soon.

I am at the point where I can push a PR that will allow the insteon_plm component to support the Insteon Hub too. This would be fully duplicative of the insteonlocal component but would add a lot of functionality to the Hub users and would mean a single component to support Insteon. The insteon_plm and insteonlocal names denote the Insteon Modem technology in use and reflect what other automation systems, like OpenHAB, have done.

The options that I see are:

  1. Depreciate the insteon_plm and insteonlocal components and create one insteon component. That will be a breaking change for all Insteon users but would make it easier for future users to know how to activate Insteon technology.
  2. Keep both insteon_plm and insteonlocal with the only difference between the two being the configuration.yaml settings to connect. This will be easier for the current users and for users coming from other technologies that take a similar approach but will lead to two components to maintain vs one.
  3. Keep insteon_plm and depreciate insteonlocal since it is maintained and the other is not. This will disrupt half of the users and have one component but will be confusing to new users who may assume the Hub is not supported based on the name unless they dig further.

My preference is to bit the bullet and go with Option 1 but before I disrupted all Insteon users I am hoping to get some input.

I’m fine with option 1.

I would go with option 1, but that’s because it won’t be too difficult for me to execute the change, and I would think that having a single supported platform ultimately benefits us as users since it makes your life easier.

I agree with option 1. I’m having trouble with the current insteonlocal component being quite buggy. Hopefully this new component will have solid performance. :grinning:

I have been testing it and the plm version is much more reliable with the hub.

Thanks @camrun91.

I have posted the PR

This PR has been merged. The Hub and the PLM now support the same devices. This includes the following HA platforms:

  • Binary Sensor: Motion, leak, door, I/O LInc sensor, other on/off sensors
  • Sensor: Smokebridge, other variable sensors
  • Lights: Almost all dimmable devices including KeypadLinc (see below for limitations)
  • Switch: Almost all on/off switch devices including I/O Linc and On/Off outlets and Keypad Linc (see below for limitations)
  • Fan: FanLinc
  • Cover: Micro Open/Close (coming in 0.77 or 0.78)
  • Thermostat: (coming in 0.78)

Limitations:

  • KeypadLinc only follows the button lights it does not activate the button scene. This should be coming soon via automations however.
  • Mini Remote - Does not show as an HA entity. It is available via automation only.
  • Hub - It may disconnect from the Hub and require both HA and the Hub to be restarted

HA documentation is here (when it gets released):

A technical list of supported devices is here:

Any device that does not have “None” as the last field shoudl work in HA (except thermostat and cover which work in the underlying library but not active in HA yet.)

This is a good idea, but will cause me some issues since I have both a Hub and a PLM connected in my setup. I will likely have to get rid of one or the other. Hummm…

@Madelinot I have created a new thread regarding the feature to allow the Hub and PLM to coexist. This is a non-trivial development activity however there has been a second request for this feature. I would like to start a discussion as to what this should include.
Insteon PLM and Hub coexist