BenPru / luxtronik

Luxtronik integration for Home Assistant
MIT License
74 stars 28 forks source link

Will it never be possible to config in yaml my sensors in 2023.9.12-Alpha-Core-Preview #129

Closed colognebroker closed 11 months ago

colognebroker commented 1 year ago

I allways configured in configuration.yaml some luxtronik devices in addition ! In the Alpha core it is not possible, also in future ?

BenPru commented 1 year ago

No. The ha guys want all configs as GUI config. Which Sensors do you need? We can implement them her as normal sensor. Or you create them and create a pr: https://github.com/BenPru/luxtronik/blob/core-preview/custom_components/luxtronik/sensor_entities_predefined.py

colognebroker commented 1 year ago

ID_WEB_Durchfluss_WQ

Am 16.09.2023 um 15:09 schrieb BenPru @.***>:

No. The ha guys want all configs as GUI config. Which Sensors do you need? We can implement them her as normal sensor. Or you create them and create a pr: https://github.com/BenPru/luxtronik/blob/core-preview/custom_components/luxtronik/sensor_entities_predefined.py

— Reply to this email directly, view it on GitHub https://github.com/BenPru/luxtronik/issues/129#issuecomment-1722227161, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFSVZLTOZUCJFPTPIKTD553X2WQJ3ANCNFSM6AAAAAA42YDPYM. You are receiving this because you authored the thread.

colognebroker commented 1 year ago

ID_WEB_Durchfluss_WQ This is flow per liter from sole pump

Am 16.09.2023 um 15:09 schrieb BenPru @.***>:

No. The ha guys want all configs as GUI config. Which Sensors do you need? We can implement them her as normal sensor. Or you create them and create a pr: https://github.com/BenPru/luxtronik/blob/core-preview/custom_components/luxtronik/sensor_entities_predefined.py

— Reply to this email directly, view it on GitHub https://github.com/BenPru/luxtronik/issues/129#issuecomment-1722227161, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFSVZLTOZUCJFPTPIKTD553X2WQJ3ANCNFSM6AAAAAA42YDPYM. You are receiving this because you authored the thread.

Kars-de-Jong commented 1 year ago

Actually, this is the flow in l/h in the heating circuit (HUP), not the VBO circuit (at least on my AIT And yes, I would like to see this sensor by default.

I have several more sensors configured, relating to the temperature delta's of the regulation.

    - name: "VBO delta calculated"
      unique_id: "ait_hup_vbo_calculated"
      state: "{{ states('sensor.luxtronik2_unknown_calculation_239')|float / 10 }}"
      availability: "{{ states('sensor.luxtronik2_unknown_calculation_239') != None }}"
      unit_of_measurement: K
      state_class: measurement
      icon: mdi:delta
    - name: "VBO delta"
      unique_id: "ait_vbo_delta"
      state: "{{ states('sensor.luxtronik2_unknown_calculation_240')|float / 10 }}"
      availability: "{{ states('sensor.luxtronik2_unknown_calculation_240') != None }}"
      unit_of_measurement: K
      state_class: measurement
      icon: mdi:delta
    - name: "HUP delta calculated"
      unique_id: "ait_hup_delta_calculated"
      state: "{{ states('sensor.luxtronik2_unknown_calculation_242')|float / 10 }}"
      availability: "{{ states('sensor.luxtronik2_unknown_calculation_242') != None }}"
      unit_of_measurement: K
      state_class: measurement
      icon: mdi:delta
    - name: "HUP delta"
      unique_id: "ait_hup_delta"
      state: "{{ states('sensor.luxtronik2_unknown_calculation_243')|float / 10 }}"
      availability: "{{ states('sensor.luxtronik2_unknown_calculation_243') != None }}"
      unit_of_measurement: K
      state_class: measurement
      icon: mdi:delta

As you can see I am using template sensors because I have to convert the raw values to float and divide the value by 10 to get the correct values, because the luxtronik Python package doesn't support them. They should really be added there, and given a proper name and type.

Looks like they were added in the latest version (not released yet):

            239: Kelvin("VBO_Temp_Spread_Soll"),
            240: Kelvin("VBO_Temp_Spread_Ist"),
            241: Percent2("HUP_PWM"),
            242: Kelvin("HUP_Temp_Spread_Soll"),
            243: Kelvin("HUP_Temp_Spread_Ist"),
BenPru commented 11 months ago
            239: Kelvin("VBO_Temp_Spread_Soll"),
            240: Kelvin("VBO_Temp_Spread_Ist"),
            241: Percent2("HUP_PWM"),
            242: Kelvin("HUP_Temp_Spread_Soll"),
            243: Kelvin("HUP_Temp_Spread_Ist"),

@Kars-de-Jong Can you provide more informations for the sensors? English key, english name, state_class, device_class, icon

BenPru commented 11 months ago

The question is answered, I close this issue. For new sensor please open another issue.