nugget / python-insteonplm

Python 3 asyncio module for interfacing with Insteon Powerline modems
MIT License
33 stars 19 forks source link

Possible bug in with Keypad 2334_222 6 button map #111

Closed wonderslug closed 5 years ago

wonderslug commented 5 years ago

Looks like the current mapping for the 6 button mode on the 2334_222 is

{3: 'C', 4: 'D', 5: 'E', 6: 'F', 7: 'G'}

But the naming convention is like in the owners manual PDF

Button Layout

I am wondering if it should be the following on the button mapping for 6?

{3: 'A', 4: 'B', 5: 'C', 6: 'D'}

It also showing a group id 7 map, does that do anything on a 2334_222 in 6 mode?

teharris1 commented 5 years ago

Yes, you are right. I only have an 8 button so I never noticed the 6 button lables and just assumed they were similar. I did choose to label "Main" as "A" instead but perhaps it should be "Main" so as not to be confusing.

I am not sure what you are refering to with the question

It also showing a group id 7 map, does that do anything on a 2334_222 in 6 mode?

Can you explain that a little better?

wonderslug commented 5 years ago

Sure I was just wondering if the group id of 7 on the 2334_222 when it was in a 6 button configuration did something specific since it was listed in the dict of button labels.

teharris1 commented 5 years ago

Closing due to PR being merged. Will be released in 0.15.0.