Skip to content

Changes to show entities for Inovelli VZM32-SN mmWave Presence Sensing Switch#597

Open
InovelliUSA wants to merge 15 commits intozigpy:devfrom
InovelliUSA:inovelli-vzm32
Open

Changes to show entities for Inovelli VZM32-SN mmWave Presence Sensing Switch#597
InovelliUSA wants to merge 15 commits intozigpy:devfrom
InovelliUSA:inovelli-vzm32

Conversation

@InovelliUSA
Copy link
Copy Markdown

No description provided.

@TheJulianJES TheJulianJES self-assigned this Dec 9, 2025
@InovelliUSA
Copy link
Copy Markdown
Author

Is the error in "Checks" because the quirk hasn't been merged yet?:
zigpy/zha-device-handlers#4552

Error:
FAILED tests/test_cluster_handlers.py::test_cluster_handler_registry - AssertionError: assert 64562 in {0: {None, 'tuya.plug_on_off_attributes', 'xiaomi.aqara_vibration_aq1', 'tuya.plug_manufacturer_attributes', 'konke.bu...e, 'tuya.plug_on_off_attributes', 'xiaomi.aqara_vibration_aq1', 'konke.button_remote', 'danfoss.ally_thermostat'}, ...}
Error: Process completed with exit code 1.

@jetpacktuxedo
Copy link
Copy Markdown

Does this need to be updated now that zigpy/zha-device-handlers#4552 merged? Or does the work to pull inovelli stuff out of base and into quirks need to happen first?

"periodic_power_and_energy_reports": True,
"active_energy_reports": True,
"power_type": False,
"switch_type": True,
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This appears to be missing quick_start_time and quick_start_level. See also: zigpy/zha-device-handlers#4821.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@wbyoung thanks for pointing that out. I think I will wait until your change merges or if this one merges I will create a new PR to add those options at that time.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@InovelliUSA FYI, zigpy/zha-device-handlers#4821 was merged along with #706.

It'd be safe to merge InovelliUSA#1 into this now (or wait as you suggested and make a follow-up PR).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants