denpamusic / homeassistant-plum-ecomax

Plum ecoMAX boiler controller integration for Home Assistant.
MIT License
29 stars 7 forks source link

Does not detect any devices after upgrade to v0.4.1 [Ecomax 850i] #56

Closed zASSAz closed 6 months ago

zASSAz commented 6 months ago

Is there an existing issue for this?

I'm having the following issue:

изображение

I have following devices connected:

I'm connecting to my devices using:

Ethernet/WiFi to RS-485 converter

I'm seeing following log messages:

2023-12-17 12:35:33.284 ERROR (MainThread) [homeassistant] Error doing job: Task exception was never retrieved
Traceback (most recent call last):
  File "/usr/local/lib/python3.11/site-packages/pyplumio/helpers/event_manager.py", line 130, in dispatch
    return_value = await callback(value)
                   ^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/pyplumio/devices/ecomax.py", line 213, in _handle_ecomax_parameters
    parameter.values = values
    ^^^^^^^^^^^^^^^^
AttributeError: 'bool' object has no attribute 'values'

My diagnostics data:

No response

Code of Conduct

denpamusic commented 6 months ago

Hi,

Thank you for the feedback!

It seems like there is name collision between circulation_pump sensor here and circulation_pump parameter here. This will affect any device, which advertises circulation pump control.

I've pulled down v0.4.1 release for now, as this is pretty easy to fix. I'll put it back up later today.

Sorry for the inconvenience!

denpamusic commented 6 months ago

This has now been fixed and test case has been added to prevent it from happening in the future. Thanks for your help!