manufacturer en modelid ophalen uit device_registry ipv bridge

[Mariusthvdb]

HI Bram,

in een test fase van de Hue API ben ik met Marcel bezig een en ander te testen. Powercalc logt het volgende:

2021-11-05 12:25:14 ERROR (MainThread) [homeassistant.config] Platform error: sensor
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/config.py", line 887, in async_process_component_config
    platform = p_integration.get_platform(domain)
  File "/usr/src/homeassistant/homeassistant/loader.py", line 530, in get_platform
    cache[full_name] = self._import_platform(platform_name)
  File "/usr/src/homeassistant/homeassistant/loader.py", line 535, in _import_platform
    return importlib.import_module(f"{self.pkg_path}.{platform_name}")
  File "/usr/local/lib/python3.9/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1030, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1007, in _find_and_load
  File "<frozen importlib._bootstrap>", line 986, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 680, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 850, in exec_module
  File "<frozen importlib._bootstrap>", line 228, in _call_with_frames_removed
  File "/config/custom_components/powercalc/sensor.py", line 65, in <module>
    from .sensors.energy import VirtualEnergySensor, create_energy_sensor
  File "/config/custom_components/powercalc/sensors/energy.py", line 23, in <module>
    from .power import VirtualPowerSensor
  File "/config/custom_components/powercalc/sensors/power.py", line 53, in <module>
    from custom_components.powercalc.model_discovery import get_light_model
  File "/config/custom_components/powercalc/model_discovery.py", line 11, in <module>
    from homeassistant.components.hue.const import DOMAIN as HUE_DOMAIN
  File "/usr/src/homeassistant/homeassistant/components/hue/__init__.py", line 13, in <module>
    from .bridge import HueBridge
  File "/usr/src/homeassistant/homeassistant/components/hue/bridge.py", line 31, in <module>
    from .sensor_base import SensorManager
  File "/usr/src/homeassistant/homeassistant/components/hue/sensor_base.py", line 9, in <module>
    from aiohue.sensors import TYPE_ZLL_PRESENCE
ModuleNotFoundError: No module named 'aiohue.sensors'

Powercalc doet dit:

homeassistant-powercalc/custom_components/powercalc/model_discovery.py

Line 70 in 6aefe0e

async def find_hue_light(

maar volgens Marcel is het beter als je rechstreeks uit de device_registry de gegevens van manufacturer en modelid ophaalt.

Zo min mogelijk de Hue bridge benaderen is het devies...

kan je er eens naar kijken?

thx!
Marijn

[marcelveldt]

Ping me maar even op discord als je hulp nodig hebt.

[bramstroker]

Hey guys,

Kan me herinneren dat ik dat wel geprobeerd heb initieel toen ik de integratie aan het ontwikkelen was, maar dat niet alle informatie daar present was (device registry). Maar kan allicht even een nieuwe blik werpen.

Het is wel een vreemde fout want de Hue component lijkt hier naar een component uit aiohue te verwijzen dat niet meer bestaat.

Hoe/waarom ben je precies de Hue API testen? Voor een nieuwe ontwikkeling daarin (beta test) oid?

[Mariusthvdb]

praktijk nu:

                "manufacturer": "Signify Netherlands B.V.",
                "model": "Hue white lamp (LWB010)",
                "name": "Garden terrace floodlight",
                "sw_version": "1.88.1",

                "manufacturer": "Signify Netherlands B.V.",
                "model": "Hue ambiance spot (LTG002)",
                "name": "Terrace spot 1",
                "sw_version": "1.88.2",

[bramstroker]

precies.... ik test graag de nieuwe integratie, maar heb nu dus geen power calc readings..... desnoods een manuele fix wordt gewaardeerd!

Het is niet 1 regeltje code. Moet even kijken of ik dit weekend 1 a 2 uurtjes kan vinden om e.e.a. te refactoren, zodat het met de device registry gaat werken. Maar moet dan zelf ook de nieuwe hue component versie hebben om het te ontwikkelen. @marcelveldt in welke branch zit dit?

[marcelveldt]

Helaas niet want dat zit nu niet in het model van de deviceregistry. Die kent maar 1 veld voor het model.
Ik kan daar het officiele model_id invullen maar dat is een beetje nietszeggende naam in de UI.

Dus vrees toch dat het een regex wordt.
Desnoods maak je het nu zo dat je eerst naar dev registry kijkt en als dat echt faalt je dan pas de bridge info zelf gaat uitlezen.
Maar ben gewaarschuwd dat de hele library op z'n kop gaat en we straks het concept kennen van V1 en V2 bridges

[marcelveldt]

precies.... ik test graag de nieuwe integratie, maar heb nu dus geen power calc readings..... desnoods een manuele fix wordt gewaardeerd!

Het is niet 1 regeltje code. Moet even kijken of ik dit weekend 1 a 2 uurtjes kan vinden om e.e.a. te refactoren, zodat het met de device registry gaat werken. Maar moet dan zelf ook de nieuwe hue component versie hebben om het te ontwikkelen. @marcelveldt in welke branch zit dit?

Heb je discord ? Dan stuur ik je even een zipje die je als custom component kunt uittesten.
Het bevind zich nu allemaal nog in de draft status en Marius is even aan het testen omdat hij een lekker complexe Hue setup heeft met meerdere bridges.

[bramstroker]

Heb je PM gestuurd op discord.

[Mariusthvdb]

dit is prachtig, gedeelde kennis maakt macht ;-)
hoop dat het lukt, en zoals altijd: beschikbaar om te testen!

[bramstroker]

@Mariusthvdb Heb vandaag aanpassingen gedaan in de code, en e.e.a. afgestemd met @marcelveldt. Bij mij werkt het nu prima met nieuwe Hue integratie die marcel aan het ontwikkelen is. Heb zojuist de PR #350 gemerged in master. Kan jij ook testen met deze versie?

[bramstroker]

Hmm vreemd. Had je die errors met die lookup tables nog? Zo ja, dan is dat de oorzaak.
De fix daarvoor zou in versie 273 (zip) van Marcel moeten zitten. Je kan anders even bij een van die lampen bij developer toolstools checken bij supported_color_modes. Staat daar brightness bij dan is dat het probleem

[Mariusthvdb]

gevonden:

en

na de update heeft the integratie allemaal nieuwe entities aangemaakt met suffix_2, en in mijn frontend staat alles nog zonder suffix natuurlijk zoals het hoort. zal die verkeerde entities weggooien en herstarten om te zien of ze weg blijven ;-)

[bramstroker]

Was het nog gelukt? Volgens mij had ik soort gelijk gedrag 1 keer toen ik aan het testen was. Maar herstart van HA had het toen opgelost. Sindsdien niet meer gehad. Heb voor de rest geen entities verwijderd.
Wellicht kan je ook even de debug log voor powercalc aan zetten?

[Mariusthvdb]

ja is opgelost! thx, voor de snelle acties, werkt weer helemaal. Echt super hoor, lees je nu alles uit de registry, en geen Bridge connectie meer?

(zag btw dat de berekeningen van de light strip niet kloppen omdat ik vergeten was mijn eigen bestanden terug te zetten, en jouw code nu gebruik . maar dat is een andere kwestie ;-). 0 verbruik bij off is een illusie.....

[bramstroker]

Goed om te horen. Yup, powercalc gebruikt niet meer rechtstreeks de hue integratie nu, maar puur de device registry. Dat maakt het in de toekomst ook mogelijk om wellicht bij andere controllers de manufacturer/model automatisch te bepalen. Bijvoorbeeld de Deconz stick.

Over de eigen bestanden, daar heb je 2 opties voor zodat je dat buiten de custom_components directory kan neerzetten. Dan kan je gewoon de code van component overschrijven zonder dat je je eigen models verliest.
Zie ook https://github.com/bramstroker/homeassistant-powercalc#light-model-library. Je kan ze neerzetten in config/powercalc-custom-models