No data anymore in homeassistant

keshavdv / victron-hacs

Home Assistant plug-in for Victron Instant Readout compatible devices

46 stars 16 forks source link

No data anymore in homeassistant #116

Open MarkusPVHM opened 1 month ago

MarkusPVHM commented 1 month ago

After updates in home assistant, there is no transfer of data anymore. Advertisement key and addresses are correct.

rmrozinsky commented 1 month ago

All my Victron Energy devices now indicate they have failed to set up. This is a fairly recent occurrence that I would guess coincided with upgrading the HA software. The following is from my logs on one of the devices:

Logger: homeassistant.config_entries Source: config_entries.py:594 First occurred: 6:31:31 AM (2 occurrences) Last logged: 7:02:43 AM

Error setting up entry SmartBatterySense for victron_ble Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/loader.py", line 1249, in _load_platform cache[full_name] = self._import_platform(platform_name) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/usr/src/homeassistant/homeassistant/loader.py", line 1281, in _import_platform return importlib.import_module(f"{self.pkg_path}.{platform_name}") ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/usr/src/homeassistant/homeassistant/util/loop.py", line 131, in protected_loop_func return func(*args, **kwargs) ^^^^^^^^^^^^^^^^^^^^^ File "/usr/local/lib/python3.12/importlib/init.py", line 90, in import_module return _bootstrap._gcd_import(name[level:], package, level) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "", line 1387, in _gcd_import File "", line 1360, in _find_and_load File "", line 1331, in _find_and_load_unlocked File "", line 935, in _load_unlocked File "", line 995, in exec_module File "", line 488, in _call_with_frames_removed File "/config/custom_components/victron_ble/sensor.py", line 216, in PassiveBluetoothDataProcessor[Optional[Union[float, int]]]


  File "/usr/local/lib/python3.12/typing.py", line 384, in inner
    return func(*args, **kwds)
           ^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/typing.py", line 1066, in _generic_class_getitem
    _check_generic(cls, params, len(cls.__parameters__))
  File "/usr/local/lib/python3.12/site-packages/typing_extensions.py", line 2947, in _check_generic
    raise TypeError(f"Too {'many' if alen > elen else 'few'} arguments"
TypeError: Too few arguments for <class 'homeassistant.components.bluetooth.passive_update_processor.PassiveBluetoothDataProcessor'>; actual 1, expected at least 2

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 594, in async_setup
    result = await component.async_setup_entry(hass, self)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/victron_ble/__init__.py", line 36, in async_setup_entry
    await hass.config_entries.async_forward_entry_setups(entry, PLATFORMS)
  File "/usr/src/homeassistant/homeassistant/config_entries.py", line 2031, in async_forward_entry_setups
    await integration.async_get_platforms(platforms)
  File "/usr/src/homeassistant/homeassistant/loader.py", line 1162, in async_get_platforms
    import_future.result()
  File "/usr/src/homeassistant/homeassistant/loader.py", line 1150, in async_get_platforms
    platforms.update(self._load_platforms(platform_names))
                     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/loader.py", line 1075, in _load_platforms
    platform_name: self._load_platform(platform_name)
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/loader.py", line 1269, in _load_platform
    raise ImportError(
ImportError: Exception importing custom_components.victron_ble.sensor

taylor-snow33 commented 4 weeks ago

I have a pull request for a fix to the sensor.py file in how it works with HA Core's PassiveBluetoothDataProcessor. https://github.com/keshavdv/victron-hacs/pull/117/commits/72f3ef29db91438a2cd0df9f1f4ba42ea85b65b4

You can use my sensor.py file as a quick hack to get this working until merged.

wmichael3 commented 4 weeks ago

Confirmed that the new sensor.py fixed it for me - SmartSolar and SmartShunt.

MarkusPVHM commented 4 weeks ago

Thank you very much, by editing the sensor.py it is working again

rmrozinsky commented 4 weeks ago

Not sure what I'm doing wrong. I edited the sensor.py file to what was provided. The entities are still not found. The following is what is annotated in the logs:

Logger: homeassistant.config_entries Source: config_entries.py:594 First occurred: 7:01:08 PM (2 occurrences) Last logged: 7:01:08 PM

Error setting up entry SmartSolar MPPT for victron_ble Error setting up entry SmartBMV 712 for victron_ble Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/loader.py", line 1249, in _load_platform cache[full_name] = self._import_platform(platform_name) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/usr/src/homeassistant/homeassistant/loader.py", line 1281, in _import_platform return importlib.import_module(f"{self.pkg_path}.{platform_name}") ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/usr/src/homeassistant/homeassistant/util/loop.py", line 131, in protected_loop_func return func(*args, **kwargs) ^^^^^^^^^^^^^^^^^^^^^ File "/usr/local/lib/python3.12/importlib/init.py", line 90, in import_module return _bootstrap._gcd_import(name[level:], package, level) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "", line 1387, in _gcd_import File "", line 1360, in _find_and_load File "", line 1331, in _find_and_load_unlocked File "", line 935, in _load_unlocked File "", line 991, in exec_module File "", line 1129, in get_code File "", line 1059, in source_to_code File "", line 488, in _call_with_frames_removed File "/config/custom_components/victron_ble/sensor.py", line 198 PassiveBluetoothDataProcessor[Optional[Union[float, int]]] ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ SyntaxError: invalid syntax. Perhaps you forgot a comma?

The above exception was the direct cause of the following exception:

Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/config_entries.py", line 594, in async_setup result = await component.async_setup_entry(hass, self) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/config/custom_components/victron_ble/init.py", line 36, in async_setup_entry await hass.config_entries.async_forward_entry_setups(entry, PLATFORMS) File "/usr/src/homeassistant/homeassistant/config_entries.py", line 2031, in async_forward_entry_setups await integration.async_get_platforms(platforms) File "/usr/src/homeassistant/homeassistant/loader.py", line 1180, in async_get_platforms platforms[platform_name] = await future ^^^^^^^^^^^^ ImportError: Exception importing custom_components.victron_ble.sensor

On Sat, Jun 8, 2024 at 3:48 PM MarkusPVHM @.***> wrote:

Thank you very much, by editing the sensor.py it is working again

— Reply to this email directly, view it on GitHub https://github.com/keshavdv/victron-hacs/issues/116#issuecomment-2156177892, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIMEU7ENR753CCE2J3DLJODZGNUZNAVCNFSM6AAAAABI7ZZFM2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJWGE3TOOBZGI . You are receiving this because you commented.Message ID: @.***>

taylor-snow33 commented 4 weeks ago

Not sure what I'm doing wrong. I edited the sensor.py file to what was

provided. The entities are still not found. The following is what is

annotated in the logs:

Logger: homeassistant.config_entries

Source: config_entries.py:594

First occurred: 7:01:08 PM (2 occurrences)

Last logged: 7:01:08 PM

Error setting up entry SmartSolar MPPT for victron_ble

Error setting up entry SmartBMV 712 for victron_ble

Traceback (most recent call last):

File "/usr/src/homeassistant/homeassistant/loader.py", line 1249, in

_load_platform
cache[full_name] = self._import_platform(platform_name)

                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/loader.py", line 1281, in

_import_platform
return importlib.import_module(f"{self.pkg_path}.{platform_name}")

       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/util/loop.py", line 131, in

protected_loop_func
return func(*args, **kwargs)

       ^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/importlib/init.py", line 90, in

import_module
return _bootstrap._gcd_import(name[level:], package, level)

       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "", line 1387, in _gcd_import

File "", line 1360, in _find_and_load

File "", line 1331, in

_find_and_load_unlocked

File "", line 935, in _load_unlocked

File "", line 991, in exec_module

File "", line 1129, in get_code

File "", line 1059, in

source_to_code

File "", line 488, in

_call_with_frames_removed

File "/config/custom_components/victron_ble/sensor.py", line 198
PassiveBluetoothDataProcessor[Optional[Union[float, int]]]

^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
SyntaxError: invalid syntax. Perhaps you forgot a comma?

The above exception was the direct cause of the following exception:

Traceback (most recent call last):

File "/usr/src/homeassistant/homeassistant/config_entries.py", line 594,

in async_setup
result = await component.async_setup_entry(hass, self)

         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/victron_ble/init.py", line 36, in

async_setup_entry
await hass.config_entries.async_forward_entry_setups(entry, PLATFORMS)
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 2031,

in async_forward_entry_setups
await integration.async_get_platforms(platforms)
File "/usr/src/homeassistant/homeassistant/loader.py", line 1180, in

async_get_platforms
platforms[platform_name] = await future

                           ^^^^^^^^^^^^
ImportError: Exception importing custom_components.victron_ble.sensor

On Sat, Jun 8, 2024 at 3:48 PM MarkusPVHM @.***> wrote:

Thank you very much, by editing the sensor.py it is working again

—

Reply to this email directly, view it on GitHub

https://github.com/keshavdv/victron-hacs/issues/116#issuecomment-2156177892,

or unsubscribe

https://github.com/notifications/unsubscribe-auth/AIMEU7ENR753CCE2J3DLJODZGNUZNAVCNFSM6AAAAABI7ZZFM2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJWGE3TOOBZGI

.

You are receiving this because you commented.Message ID:

@.***>

It appears you have a syntax error or missing comma. Double check your copy paste of the file. Should be fine but happy to look further in the morning if needed.

rmrozinsky commented 4 weeks ago

Thanks, I appreciate the help. I'm doing a copy and paste from the file provided. I did find the missing comma. However, it didn't resolve the issue. Here is a copy of the error from my logs:

Logger: homeassistant.config_entries Source: config_entries.py:594 First occurred: 8:27:41 PM (2 occurrences) Last logged: 8:27:41 PM

Error setting up entry SmartSolar MPPT for victron_ble Error setting up entry SmartBMV 712 for victron_ble Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/loader.py", line 1249, in _load_platform cache[full_name] = self._import_platform(platform_name) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/usr/src/homeassistant/homeassistant/loader.py", line 1281, in _import_platform return importlib.import_module(f"{self.pkg_path}.{platform_name}") ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/usr/src/homeassistant/homeassistant/util/loop.py", line 131, in protected_loop_func return func(*args, **kwargs) ^^^^^^^^^^^^^^^^^^^^^ File "/usr/local/lib/python3.12/importlib/init.py", line 90, in import_module return _bootstrap._gcd_import(name[level:], package, level) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "", line 1387, in _gcd_import File "", line 1360, in _find_and_load File "", line 1331, in _find_and_load_unlocked File "", line 935, in _load_unlocked File "", line 995, in exec_module File "", line 488, in _call_with_frames_removed File "/config/custom_components/victron_ble/sensor.py", line 198, in

PassiveBluetoothDataProcessor[Optional[Union[float, int]]], ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/usr/local/lib/python3.12/typing.py", line 384, in inner return func(*args, **kwds) ^^^^^^^^^^^^^^^^^^^ File "/usr/local/lib/python3.12/typing.py", line 1066, in _generic_class_getitem _check_generic(cls, params, len(cls.__parameters__)) File "/usr/local/lib/python3.12/site-packages/typing_extensions.py", line 2947, in _check_generic raise TypeError(f"Too {'many' if alen > elen else 'few'} arguments" TypeError: Too few arguments for ; actual 1, expected at least 2 The above exception was the direct cause of the following exception: Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/config_entries.py", line 594, in async_setup result = await component.async_setup_entry(hass, self) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/config/custom_components/victron_ble/__init__.py", line 36, in async_setup_entry await hass.config_entries.async_forward_entry_setups(entry, PLATFORMS) File "/usr/src/homeassistant/homeassistant/config_entries.py", line 2031, in async_forward_entry_setups await integration.async_get_platforms(platforms) File "/usr/src/homeassistant/homeassistant/loader.py", line 1180, in async_get_platforms platforms[platform_name] = await future ^^^^^^^^^^^^ ImportError: Exception importing custom_components.victron_ble.sensor On Sat, Jun 8, 2024 at 8:15 PM Taylor Snow ***@***.***> wrote: > Not sure what I'm doing wrong. I edited the sensor.py file to what was > > provided. The entities are still not found. The following is what is > > annotated in the logs: > > Logger: homeassistant.config_entries > > Source: config_entries.py:594 > > First occurred: 7:01:08 PM (2 occurrences) > > Last logged: 7:01:08 PM > > Error setting up entry SmartSolar MPPT for victron_ble > > Error setting up entry SmartBMV 712 for victron_ble > > Traceback (most recent call last): > > File "/usr/src/homeassistant/homeassistant/loader.py", line 1249, in > > _load_platform > > cache[full_name] = self._import_platform(platform_name) > > ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ > > File "/usr/src/homeassistant/homeassistant/loader.py", line 1281, in > > _import_platform > > return importlib.import_module(f"{self.pkg_path}.{platform_name}") > > ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ > > File "/usr/src/homeassistant/homeassistant/util/loop.py", line 131, in > > protected_loop_func > > return func(*args, **kwargs) > > ^^^^^^^^^^^^^^^^^^^^^ > > File "/usr/local/lib/python3.12/importlib/*init*.py", line 90, in > > import_module > > return _bootstrap._gcd_import(name[level:], package, level) > > ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ > > File "", line 1387, in _gcd_import > > File "", line 1360, in _find_and_load > > File "", line 1331, in > > _find_and_load_unlocked > > File "", line 935, in _load_unlocked > > File "", line 991, in exec_module > > File "", line 1129, in get_code > > File "", line 1059, in > > source_to_code > > File "", line 488, in > > _call_with_frames_removed > > File "/config/custom_components/victron_ble/sensor.py", line 198 > > PassiveBluetoothDataProcessor[Optional[Union[float, int]]] > > ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ > > SyntaxError: invalid syntax. Perhaps you forgot a comma? > > The above exception was the direct cause of the following exception: > > Traceback (most recent call last): > > File "/usr/src/homeassistant/homeassistant/config_entries.py", line 594, > > in async_setup > > result = await component.async_setup_entry(hass, self) > > ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ > > File "/config/custom_components/victron_ble/*init*.py", line 36, in > > async_setup_entry > > await hass.config_entries.async_forward_entry_setups(entry, PLATFORMS) > > File "/usr/src/homeassistant/homeassistant/config_entries.py", line 2031, > > in async_forward_entry_setups > > await integration.async_get_platforms(platforms) > > File "/usr/src/homeassistant/homeassistant/loader.py", line 1180, in > > async_get_platforms > > platforms[platform_name] = await future > > ^^^^^^^^^^^^ > > ImportError: Exception importing custom_components.victron_ble.sensor > > On Sat, Jun 8, 2024 at 3:48 PM MarkusPVHM *@*.***> wrote: > > Thank you very much, by editing the sensor.py it is working again > > — > > Reply to this email directly, view it on GitHub > > #116 (comment) > > , > > or unsubscribe > > > https://github.com/notifications/unsubscribe-auth/AIMEU7ENR753CCE2J3DLJODZGNUZNAVCNFSM6AAAAABI7ZZFM2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJWGE3TOOBZGI > > . > > You are receiving this because you commented.Message ID: > > *@*.***> > > It appears you have a syntax error or missing comma. Double check your > copy paste of the file. Should be fine but happy to look further in the > morning if needed. > > — > Reply to this email directly, view it on GitHub > , > or unsubscribe > > . > You are receiving this because you commented.Message ID: > ***@***.***> >

rmrozinsky commented 3 weeks ago

Any chance anyone has any insight into why Taylor's sensor.py is not working for me? Seems straight forward, but I not getting good results. Basically, cut and paste Taylor's file into /homeassistant/custom_components/victron_ble/sensor.py, and restarted HA.

I'd sure like to get this working again. Thanks.

On Sat, Jun 8, 2024 at 8:40 PM Richard Mrozinsky @.***> wrote:

Thanks, I appreciate the help. I'm doing a copy and paste from the file provided. I did find the missing comma. However, it didn't resolve the issue. Here is a copy of the error from my logs:

Logger: homeassistant.config_entries Source: config_entries.py:594 First occurred: 8:27:41 PM (2 occurrences) Last logged: 8:27:41 PM

Error setting up entry SmartSolar MPPT for victron_ble Error setting up entry SmartBMV 712 for victron_ble Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/loader.py", line 1249, in _load_platform cache[full_name] = self._import_platform(platform_name) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/usr/src/homeassistant/homeassistant/loader.py", line 1281, in _import_platform return importlib.import_module(f"{self.pkg_path}.{platform_name}") ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/usr/src/homeassistant/homeassistant/util/loop.py", line 131, in protected_loop_func return func(*args, **kwargs) ^^^^^^^^^^^^^^^^^^^^^ File "/usr/local/lib/python3.12/importlib/init.py", line 90, in import_module return _bootstrap._gcd_import(name[level:], package, level) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "", line 1387, in _gcd_import File "", line 1360, in _find_and_load File "", line 1331, in _find_and_load_unlocked File "", line 935, in _load_unlocked File "", line 995, in exec_module File "", line 488, in _call_with_frames_removed File "/config/custom_components/victron_ble/sensor.py", line 198, in
PassiveBluetoothDataProcessor[Optional[Union[float, int]]], ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/usr/local/lib/python3.12/typing.py", line 384, in inner return func(*args, **kwds) ^^^^^^^^^^^^^^^^^^^ File "/usr/local/lib/python3.12/typing.py", line 1066, in _generic_class_getitem _check_generic(cls, params, len(cls.__parameters__)) File "/usr/local/lib/python3.12/site-packages/typing_extensions.py", line 2947, in _check_generic raise TypeError(f"Too {'many' if alen > elen else 'few'} arguments" TypeError: Too few arguments for ; actual 1, expected at least 2 The above exception was the direct cause of the following exception: Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/config_entries.py", line 594, in async_setup result = await component.async_setup_entry(hass, self) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/config/custom_components/victron_ble/__init__.py", line 36, in async_setup_entry await hass.config_entries.async_forward_entry_setups(entry, PLATFORMS) File "/usr/src/homeassistant/homeassistant/config_entries.py", line 2031, in async_forward_entry_setups await integration.async_get_platforms(platforms) File "/usr/src/homeassistant/homeassistant/loader.py", line 1180, in async_get_platforms platforms[platform_name] = await future ^^^^^^^^^^^^ ImportError: Exception importing custom_components.victron_ble.sensor On Sat, Jun 8, 2024 at 8:15 PM Taylor Snow ***@***.***> wrote: > Not sure what I'm doing wrong. I edited the sensor.py file to what was > > provided. The entities are still not found. The following is what is > > annotated in the logs: > > Logger: homeassistant.config_entries > > Source: config_entries.py:594 > > First occurred: 7:01:08 PM (2 occurrences) > > Last logged: 7:01:08 PM > > Error setting up entry SmartSolar MPPT for victron_ble > > Error setting up entry SmartBMV 712 for victron_ble > > Traceback (most recent call last): > > File "/usr/src/homeassistant/homeassistant/loader.py", line 1249, in > > _load_platform > > cache[full_name] = self._import_platform(platform_name) > > ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ > > File "/usr/src/homeassistant/homeassistant/loader.py", line 1281, in > > _import_platform > > return importlib.import_module(f"{self.pkg_path}.{platform_name}") > > ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ > > File "/usr/src/homeassistant/homeassistant/util/loop.py", line 131, in > > protected_loop_func > > return func(*args, **kwargs) > > ^^^^^^^^^^^^^^^^^^^^^ > > File "/usr/local/lib/python3.12/importlib/*init*.py", line 90, in > > import_module > > return _bootstrap._gcd_import(name[level:], package, level) > > ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ > > File "", line 1387, in _gcd_import > > File "", line 1360, in _find_and_load > > File "", line 1331, in > > _find_and_load_unlocked > > File "", line 935, in _load_unlocked > > File "", line 991, in exec_module > > File "", line 1129, in get_code > > File "", line 1059, in > > source_to_code > > File "", line 488, in > > _call_with_frames_removed > > File "/config/custom_components/victron_ble/sensor.py", line 198 > > PassiveBluetoothDataProcessor[Optional[Union[float, int]]] > > ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ > > SyntaxError: invalid syntax. Perhaps you forgot a comma? > > The above exception was the direct cause of the following exception: > > Traceback (most recent call last): > > File "/usr/src/homeassistant/homeassistant/config_entries.py", line 594, > > in async_setup > > result = await component.async_setup_entry(hass, self) > > ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ > > File "/config/custom_components/victron_ble/*init*.py", line 36, in > > async_setup_entry > > await hass.config_entries.async_forward_entry_setups(entry, PLATFORMS) > > File "/usr/src/homeassistant/homeassistant/config_entries.py", line 2031, > > in async_forward_entry_setups > > await integration.async_get_platforms(platforms) > > File "/usr/src/homeassistant/homeassistant/loader.py", line 1180, in > > async_get_platforms > > platforms[platform_name] = await future > > ^^^^^^^^^^^^ > > ImportError: Exception importing custom_components.victron_ble.sensor > > On Sat, Jun 8, 2024 at 3:48 PM MarkusPVHM *@*.***> wrote: > > Thank you very much, by editing the sensor.py it is working again > > — > > Reply to this email directly, view it on GitHub > > #116 (comment) > > , > > or unsubscribe > > > https://github.com/notifications/unsubscribe-auth/AIMEU7ENR753CCE2J3DLJODZGNUZNAVCNFSM6AAAAABI7ZZFM2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJWGE3TOOBZGI > > . > > You are receiving this because you commented.Message ID: > > *@*.***> > > It appears you have a syntax error or missing comma. Double check your > copy paste of the file. Should be fine but happy to look further in the > morning if needed. > > — > Reply to this email directly, view it on GitHub > , > or unsubscribe > > . > You are receiving this because you commented.Message ID: > ***@***.***> >

MarkusPVHM commented 3 weeks ago

You can not copy paste the file. You have to edit your original file, remove or add the relevant lines there... Von meinem HONOR-Telefon gesendet-------- Ursprüngliche Nachricht --------Von: rmrozinsky @.>Datum: Di., 11. Juni 2024, 03:04An: keshavdv/victron-hacs @.>Cc: MarkusPVHM @.>, Author @.>Betreff: Re: [keshavdv/victron-hacs] No data anymore in homeassistant (Issue #116) Any chance anyone has any insight into why Taylor's sensor.py is not

working for me? Seems straight forward, but I not getting good results.

Basically, cut and paste Taylor's file

into /homeassistant/custom_components/victron_ble/sensor.py, and restarted

HA.

I'd sure like to get this working again. Thanks.

On Sat, Jun 8, 2024 at 8:40 PM Richard Mrozinsky @.***>

wrote:

Thanks, I appreciate the help. I'm doing a copy and paste from the file

provided. I did find the missing comma. However, it didn't resolve the

issue. Here is a copy of the error from my logs:

Logger: homeassistant.config_entries

Source: config_entries.py:594

First occurred: 8:27:41 PM (2 occurrences)

Last logged: 8:27:41 PM

Error setting up entry SmartSolar MPPT for victron_ble

Error setting up entry SmartBMV 712 for victron_ble

Traceback (most recent call last):

File "/usr/src/homeassistant/homeassistant/loader.py", line 1249, in

_load_platform
cache[full_name] = self._import_platform(platform_name)

                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/loader.py", line 1281, in

_import_platform
return importlib.import_module(f"{self.pkg_path}.{platform_name}")

       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/util/loop.py", line 131, in

protected_loop_func
return func(*args, **kwargs)

       ^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/importlib/init.py", line 90, in

import_module
return _bootstrap._gcd_import(name[level:], package, level)

       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "", line 1387, in _gcd_import

File "", line 1360, in _find_and_load

File "", line 1331, in

_find_and_load_unlocked

File "", line 935, in _load_unlocked

File "", line 995, in exec_module

File "", line 488, in

_call_with_frames_removed

File "/config/custom_components/victron_ble/sensor.py", line 198, in
PassiveBluetoothDataProcessor[Optional[Union[float, int]]], ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/usr/local/lib/python3.12/typing.py", line 384, in inner return func(*args, **kwds) ^^^^^^^^^^^^^^^^^^^ File "/usr/local/lib/python3.12/typing.py", line 1066, in _generic_class_getitem _check_generic(cls, params, len(cls.__parameters__)) File "/usr/local/lib/python3.12/site-packages/typing_extensions.py", line 2947, in _check_generic raise TypeError(f"Too {'many' if alen > elen else 'few'} arguments" TypeError: Too few arguments for ; actual 1, expected at least 2 The above exception was the direct cause of the following exception: Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/config_entries.py", line 594, in async_setup result = await component.async_setup_entry(hass, self) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/config/custom_components/victron_ble/__init__.py", line 36, in async_setup_entry await hass.config_entries.async_forward_entry_setups(entry, PLATFORMS) File "/usr/src/homeassistant/homeassistant/config_entries.py", line 2031, in async_forward_entry_setups await integration.async_get_platforms(platforms) File "/usr/src/homeassistant/homeassistant/loader.py", line 1180, in async_get_platforms platforms[platform_name] = await future ^^^^^^^^^^^^ ImportError: Exception importing custom_components.victron_ble.sensor On Sat, Jun 8, 2024 at 8:15 PM Taylor Snow ***@***.***> wrote: > Not sure what I'm doing wrong. I edited the sensor.py file to what was > > provided. The entities are still not found. The following is what is > > annotated in the logs: > > Logger: homeassistant.config_entries > > Source: config_entries.py:594 > > First occurred: 7:01:08 PM (2 occurrences) > > Last logged: 7:01:08 PM > > Error setting up entry SmartSolar MPPT for victron_ble > > Error setting up entry SmartBMV 712 for victron_ble > > Traceback (most recent call last): > > File "/usr/src/homeassistant/homeassistant/loader.py", line 1249, in > > _load_platform > > cache[full_name] = self._import_platform(platform_name) > > ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ > > File "/usr/src/homeassistant/homeassistant/loader.py", line 1281, in > > _import_platform > > return importlib.import_module(f"{self.pkg_path}.{platform_name}") > > ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ > > File "/usr/src/homeassistant/homeassistant/util/loop.py", line 131, in > > protected_loop_func > > return func(*args, **kwargs) > > ^^^^^^^^^^^^^^^^^^^^^ > > File "/usr/local/lib/python3.12/importlib/*init*.py", line 90, in > > import_module > > return _bootstrap._gcd_import(name[level:], package, level) > > ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ > > File "", line 1387, in _gcd_import > > File "", line 1360, in _find_and_load > > File "", line 1331, in > > _find_and_load_unlocked > > File "", line 935, in _load_unlocked > > File "", line 991, in exec_module > > File "", line 1129, in get_code > > File "", line 1059, in > > source_to_code > > File "", line 488, in > > _call_with_frames_removed > > File "/config/custom_components/victron_ble/sensor.py", line 198 > > PassiveBluetoothDataProcessor[Optional[Union[float, int]]] > > ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ > > SyntaxError: invalid syntax. Perhaps you forgot a comma? > > The above exception was the direct cause of the following exception: > > Traceback (most recent call last): > > File "/usr/src/homeassistant/homeassistant/config_entries.py", line 594, > > in async_setup > > result = await component.async_setup_entry(hass, self) > > ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ > > File "/config/custom_components/victron_ble/*init*.py", line 36, in > > async_setup_entry > > await hass.config_entries.async_forward_entry_setups(entry, PLATFORMS) > > File "/usr/src/homeassistant/homeassistant/config_entries.py", line 2031, > > in async_forward_entry_setups > > await integration.async_get_platforms(platforms) > > File "/usr/src/homeassistant/homeassistant/loader.py", line 1180, in > > async_get_platforms > > platforms[platform_name] = await future > > ^^^^^^^^^^^^ > > ImportError: Exception importing custom_components.victron_ble.sensor > > On Sat, Jun 8, 2024 at 3:48 PM MarkusPVHM *@*.***> wrote: > > Thank you very much, by editing the sensor.py it is working again > > — > > Reply to this email directly, view it on GitHub > > #116 (comment) > > , > > or unsubscribe > > > https://github.com/notifications/unsubscribe-auth/AIMEU7ENR753CCE2J3DLJODZGNUZNAVCNFSM6AAAAABI7ZZFM2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJWGE3TOOBZGI > > . > > You are receiving this because you commented.Message ID: > > *@*.***> > > It appears you have a syntax error or missing comma. Double check your > copy paste of the file. Should be fine but happy to look further in the > morning if needed. > > — > Reply to this email directly, view it on GitHub > , > or unsubscribe > > . > You are receiving this because you commented.Message ID: > ***@***.***> >

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

rmrozinsky commented 3 weeks ago

Thanks, learning something new all the time. I restored the previous version of HA because I felt I really screwed up the sensor.py doing the full copy and paste. When I did that, all the sensors started working. Am I correct that the sensor.py fix is required because of a conflict with HA Core 2024.6.1? I think I'll leave things as they are for now and wait until the new sensor.py file is merged. How will I be able to tell when that happens?

Thanks for the hard work. The ability to monitor the Victron sensors remotely is very helpful.

On Tue, Jun 11, 2024 at 12:25 AM MarkusPVHM @.***> wrote:

You can not copy paste the file. You have to edit your original file, remove or add the relevant lines there... Von meinem HONOR-Telefon gesendet-------- Ursprüngliche Nachricht --------Von: rmrozinsky @.>Datum: Di., 11. Juni 2024, 03:04An: keshavdv/victron-hacs @.>Cc: MarkusPVHM @.>, Author @.>Betreff: Re: [keshavdv/victron-hacs] No data anymore in homeassistant (Issue #116) Any chance anyone has any insight into why Taylor's sensor.py is not

working for me? Seems straight forward, but I not getting good results.

Basically, cut and paste Taylor's file

into /homeassistant/custom_components/victron_ble/sensor.py, and restarted

HA.

I'd sure like to get this working again. Thanks.

On Sat, Jun 8, 2024 at 8:40 PM Richard Mrozinsky @.***>

wrote:

Thanks, I appreciate the help. I'm doing a copy and paste from the file

provided. I did find the missing comma. However, it didn't resolve the

issue. Here is a copy of the error from my logs:

Logger: homeassistant.config_entries

Source: config_entries.py:594

First occurred: 8:27:41 PM (2 occurrences)

Last logged: 8:27:41 PM

Error setting up entry SmartSolar MPPT for victron_ble

Error setting up entry SmartBMV 712 for victron_ble

Traceback (most recent call last):

File "/usr/src/homeassistant/homeassistant/loader.py", line 1249, in

_load_platform

cache[full_name] = self._import_platform(platform_name)

^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

File "/usr/src/homeassistant/homeassistant/loader.py", line 1281, in

_import_platform

return importlib.import_module(f"{self.pkg_path}.{platform_name}")

^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

File "/usr/src/homeassistant/homeassistant/util/loop.py", line 131, in

protected_loop_func

return func(*args, **kwargs)

^^^^^^^^^^^^^^^^^^^^^

File "/usr/local/lib/python3.12/importlib/init.py", line 90, in

import_module

return _bootstrap._gcd_import(name[level:], package, level)

^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

File "", line 1387, in _gcd_import

File "", line 1360, in _find_and_load

File "", line 1331, in

_find_and_load_unlocked

File "", line 935, in _load_unlocked

File "", line 995, in exec_module

File "", line 488, in

_call_with_frames_removed

File "/config/custom_components/victron_ble/sensor.py", line 198, in
PassiveBluetoothDataProcessor[Optional[Union[float, int]]], ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/usr/local/lib/python3.12/typing.py", line 384, in inner return func(*args, **kwds) ^^^^^^^^^^^^^^^^^^^ File "/usr/local/lib/python3.12/typing.py", line 1066, in _generic_class_getitem _check_generic(cls, params, len(cls.__parameters__)) File "/usr/local/lib/python3.12/site-packages/typing_extensions.py", line 2947, in _check_generic raise TypeError(f"Too {'many' if alen > elen else 'few'} arguments" TypeError: Too few arguments for ; actual 1, expected at least 2 The above exception was the direct cause of the following exception: Traceback (most recent call last): File "/usr/src/homeassistant/homeassistant/config_entries.py", line 594, in async_setup result = await component.async_setup_entry(hass, self) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ File "/config/custom_components/victron_ble/__init__.py", line 36, in async_setup_entry await hass.config_entries.async_forward_entry_setups(entry, PLATFORMS) File "/usr/src/homeassistant/homeassistant/config_entries.py", line 2031, in async_forward_entry_setups await integration.async_get_platforms(platforms) File "/usr/src/homeassistant/homeassistant/loader.py", line 1180, in async_get_platforms platforms[platform_name] = await future ^^^^^^^^^^^^ ImportError: Exception importing custom_components.victron_ble.sensor On Sat, Jun 8, 2024 at 8:15 PM Taylor Snow ***@***.***> wrote: > Not sure what I'm doing wrong. I edited the sensor.py file to what was > > provided. The entities are still not found. The following is what is > > annotated in the logs: > > Logger: homeassistant.config_entries > > Source: config_entries.py:594 > > First occurred: 7:01:08 PM (2 occurrences) > > Last logged: 7:01:08 PM > > Error setting up entry SmartSolar MPPT for victron_ble > > Error setting up entry SmartBMV 712 for victron_ble > > Traceback (most recent call last): > > File "/usr/src/homeassistant/homeassistant/loader.py", line 1249, in > > _load_platform > > cache[full_name] = self._import_platform(platform_name) > > ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ > > File "/usr/src/homeassistant/homeassistant/loader.py", line 1281, in > > _import_platform > > return importlib.import_module(f"{self.pkg_path}.{platform_name}") > > ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ > > File "/usr/src/homeassistant/homeassistant/util/loop.py", line 131, in > > protected_loop_func > > return func(*args, **kwargs) > > ^^^^^^^^^^^^^^^^^^^^^ > > File "/usr/local/lib/python3.12/importlib/*init*.py", line 90, in > > import_module > > return _bootstrap._gcd_import(name[level:], package, level) > > ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ > > File "", line 1387, in _gcd_import > > File "", line 1360, in _find_and_load > > File "", line 1331, in > > _find_and_load_unlocked > > File "", line 935, in _load_unlocked > > File "", line 991, in exec_module > > File "", line 1129, in get_code > > File "", line 1059, in > > source_to_code > > File "", line 488, in > > _call_with_frames_removed > > File "/config/custom_components/victron_ble/sensor.py", line 198 > > PassiveBluetoothDataProcessor[Optional[Union[float, int]]] > > ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ > > SyntaxError: invalid syntax. Perhaps you forgot a comma? > > The above exception was the direct cause of the following exception: > > Traceback (most recent call last): > > File "/usr/src/homeassistant/homeassistant/config_entries.py", line 594, > > in async_setup > > result = await component.async_setup_entry(hass, self) > > ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ > > File "/config/custom_components/victron_ble/*init*.py", line 36, in > > async_setup_entry > > await hass.config_entries.async_forward_entry_setups(entry, PLATFORMS) > > File "/usr/src/homeassistant/homeassistant/config_entries.py", line 2031, > > in async_forward_entry_setups > > await integration.async_get_platforms(platforms) > > File "/usr/src/homeassistant/homeassistant/loader.py", line 1180, in > > async_get_platforms > > platforms[platform_name] = await future > > ^^^^^^^^^^^^ > > ImportError: Exception importing custom_components.victron_ble.sensor > > On Sat, Jun 8, 2024 at 3:48 PM MarkusPVHM *@*.***> wrote: > > Thank you very much, by editing the sensor.py it is working again > > — > > Reply to this email directly, view it on GitHub > > #116 (comment) > < https://github.com/keshavdv/victron-hacs/issues/116#issuecomment-2156177892> > , > > or unsubscribe > > > https://github.com/notifications/unsubscribe-auth/AIMEU7ENR753CCE2J3DLJODZGNUZNAVCNFSM6AAAAABI7ZZFM2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJWGE3TOOBZGI > > . > > You are receiving this because you commented.Message ID: > > *@*.***> > > It appears you have a syntax error or missing comma. Double check your > copy paste of the file. Should be fine but happy to look further in the > morning if needed. > > — > Reply to this email directly, view it on GitHub > < https://github.com/keshavdv/victron-hacs/issues/116#issuecomment-2156255308>, > or unsubscribe > < https://github.com/notifications/unsubscribe-auth/AIMEU7GG6D666VKE4HQVNBTZGOUDTAVCNFSM6AAAAABI7ZZFM2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJWGI2TKMZQHA> > . > You are receiving this because you commented.Message ID: > ***@***.***> >

—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

— Reply to this email directly, view it on GitHub https://github.com/keshavdv/victron-hacs/issues/116#issuecomment-2159819790, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIMEU7DJBITRFVQQ3UMNAODZG2C3RAVCNFSM6AAAAABI7ZZFM2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJZHAYTSNZZGA . You are receiving this because you commented.Message ID: @.***>

phuse commented 3 weeks ago

update fixed the data from the Smart solars and smart battery sense but not for BMV-712

JamesPerrins commented 1 week ago

Same for me - smart solars now working but BMV-712 isn't

brooksben11 commented 6 days ago

Finally

update fixed the data from the Smart solars and smart battery sense but not for BMV-712

Same for me - smart solars now working but BMV-712 isn't

Finally got around to making this change; no issues with my BMV-712 reporting data now. I assume you reloaded the integration after making the changes to the file?