Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

EcoFlow-integration stopped after update HA Core 2025.1.0 #116

Open
BartGysens opened this issue Jan 4, 2025 · 13 comments
Open

EcoFlow-integration stopped after update HA Core 2025.1.0 #116

BartGysens opened this issue Jan 4, 2025 · 13 comments

Comments

@BartGysens
Copy link

BartGysens commented Jan 4, 2025

Integration stopped working after 2025.1.0 update.

System:

  • Core 2025.1.0
  • Supervisor 2024.12.3
  • Operating System 14.1
  • Frontend 20250103.0

Error-log:

2025-01-04 15:37:03.553 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry DELTA Pro 260736 for ecoflow
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 640, in __async_setup_with_context
    result = await component.async_setup_entry(hass, self)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/ecoflow/__init__.py", line 275, in async_setup_entry
    await hass.config_entries.async_forward_entry_setups(entry, _PLATFORMS)
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 2278, in async_forward_entry_setups
    await integration.async_get_platforms(platforms)
  File "/usr/src/homeassistant/homeassistant/loader.py", line 1185, in async_get_platforms
    import_future.result()
    ~~~~~~~~~~~~~~~~~~~~^^
  File "/usr/src/homeassistant/homeassistant/loader.py", line 1185, in async_get_platforms
    import_future.result()
    ~~~~~~~~~~~~~~~~~~~~^^
  File "/usr/src/homeassistant/homeassistant/loader.py", line 1185, in async_get_platforms
    import_future.result()
    ~~~~~~~~~~~~~~~~~~~~^^
  [Previous line repeated 3 more times]
  File "/usr/src/homeassistant/homeassistant/loader.py", line 1173, in async_get_platforms
    platforms.update(self._load_platforms(platform_names))
                     ~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/loader.py", line 1098, in _load_platforms
    platform_name: self._load_platform(platform_name)
                   ~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/loader.py", line 1272, in _load_platform
    cache[full_name] = self._import_platform(platform_name)
                       ~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/loader.py", line 1304, in _import_platform
    return importlib.import_module(f"{self.pkg_path}.{platform_name}")
           ~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/util/loop.py", line 200, in protected_loop_func
    return func(*args, **kwargs)
  File "/usr/local/lib/python3.13/importlib/__init__.py", line 88, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
           ~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<frozen importlib._bootstrap>", line 1387, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1360, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1331, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 935, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 1026, in exec_module
  File "<frozen importlib._bootstrap>", line 488, in _call_with_frames_removed
  File "/config/custom_components/ecoflow/select.py", line 5, in <module>
    from homeassistant.const import FREQUENCY_HERTZ
ImportError: cannot import name 'FREQUENCY_HERTZ' from 'homeassistant.const' (/usr/src/homeassistant/homeassistant/const.py)
@thrawnis
Copy link

thrawnis commented Jan 4, 2025

Also seeing the exact same issue after the HA update

@davethetallguy
Copy link

davethetallguy commented Jan 5, 2025

I think it's worth observing that the developer has not updated this integration since mid 2022.

In addition, #115, opened in October 2024, identifies FREQUENCY_HERTZ (the start of the error above) as deprecated (removed) in HA CORE 2025.1.0. Other elements are also deprecated.

The developer did not respond to the 2024 bug, and in fact most of their repositories have been un-maintained since 2023.

While I would be happy for the original developer to patch the software, we (the active users) may need to identify someone to take over maintenance.

ETA: #108 was opened in February, also identifying the constants which were just deprecated.

The last similar bug fix for #76 was identified by @Patrick762, however @vwt12eh8 never merged the fix, which seems again to point to this integration now being unsupported.

@klevytskyi
Copy link

I've created #118 which works fine for me right now, so feel free to check it out if you want to upgrade. I hope it will be merged someday, but it seems I'll just use my fork for integration.

@davethetallguy
Copy link

Brilliant work! Thanks for the assistance. I replaced the custom repository on my host and it came right up (model: Delta Pro)

@vlad-6502
Copy link

It works for my Delta Pro! Thank you so much!

@skpow55
Copy link

skpow55 commented Jan 7, 2025 via email

@imbrianj
Copy link

imbrianj commented Jan 7, 2025

Thank you, @klevytskyi - this is now working for me for my Delta Pro and Delta Mini. Really appreciate it!

@thrawnis
Copy link

thrawnis commented Jan 7, 2025

Thank you @klevytskyi! I can confirm my two Delta Pros and an extra battery are all working properly again.

For anyone that doesn't know how to implement this fix...

  1. Remove the HACS custom repo: https://github.com/vwt12eh8/hassio-ecoflow
  2. Add the new custom repo: https://github.com/klevytskyi/hassio-ecoflow
  3. Wait for Home Assistant to prompt you to update the Ecoflow integration to the latest version and proceed
  4. Restart Home Assistant

@BartGysens
Copy link
Author

Thank you, works great +1

@Mike-DUS
Copy link

Mike-DUS commented Jan 7, 2025

Thanks for the update, works perfect.

Are there any plans to continue the development and integrate all products being on the market ?

@Solter59
Copy link

Solter59 commented Jan 7, 2025

Thanks for the update. It works fine for me with my Delta Pro. I closed #115

@claudeju
Copy link

claudeju commented Jan 8, 2025

Thank you good work

@odmyko
Copy link

odmyko commented Jan 10, 2025

Thanks!

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

No branches or pull requests