Git Product home page Git Product logo

Comments (18)

fustom avatar fustom commented on September 23, 2024 2

Ok I checked ch_flame and dhw_flame on Chomupashchuk's v2 integration. That's actually a lie.
https://github.com/chomupashchuk/ariston-remotethermo-home-assistant-v2/blob/af5b4f4d3b47901faa0c0ae0bd339e8a4f9364bc/custom_components/ariston/ariston.py#L1008

        if sensor == self._PARAM_DHW_FLAME:
            value = None
            try:
                increase_dhw_temp = None
                new_value = self._ariston_sensors[self._PARAM_DHW_STORAGE_TEMPERATURE][self._VALUE]
                if new_value:
                    increase_dhw_temp = False
                    if self._last_dhw_storage_temp is not None and \
                        self._last_dhw_storage_temp > 0 and \
                        new_value > 0:
                        if new_value > self._last_dhw_storage_temp:
                            increase_dhw_temp = True
                    elif self._last_dhw_storage_temp is not None and\
                        new_value > 0:
                        self._last_dhw_storage_temp = new_value
            except Exception:
                increase_dhw_temp = None
            ch_flame = None
            for zone in self._zones:
                ch_flame_zone = self._ariston_sensors[self._zone_sensor_name(self._PARAM_FLAME, zone)][self._VALUE]
                if ch_flame_zone in self._OFF_ON_TEXT:
                    if ch_flame is None or ch_flame == self._OFF:
                        ch_flame = ch_flame_zone
            if self._ariston_sensors[self._PARAM_FLAME][self._VALUE] in self._OFF_ON_TEXT and ch_flame in self._OFF_ON_TEXT:
                if self._ariston_sensors[self._PARAM_FLAME][self._VALUE] == self._OFF:
                    value = self._OFF
                elif ch_flame == self._OFF:
                    value = self._ON
                else:
                    # Unknown state of DHW
                    value = increase_dhw_temp
        return value

It is a horror, but as far as I understand:
For dhw_flame he checks the zones for heat requests.

  1. No zone with heat request. Checks the flame flag.
    a) The flame is true => Returns dhw_flame 'ON'
    b) The flame is false => Returns dhw_flame 'OFF'
  2. There is a zone with heat request. He check is the DHW storage temperature higher then the last stored temperature.
    a) The DHW storage temperature higher => Returns dhw_flame True
    b) The DHW storage temperature not higher => Returns dhw_flame False
    c) Could not get DHW storage temperature => Returns dhw_flame None

For ch_flame he just returns the heat request for the zone.
I can do the same. But I really do not want to do that.

from ariston-remotethermo-home-assistant-v3.

Jedrek2021 avatar Jedrek2021 commented on September 23, 2024 1

Are temperature reading errors for GENUS ONE SYSTEM caused by incorrect device identification? I can see device information that is returned to me via GALEVO
and software version: 3.5.3_110.15.83. Can another type of device be forced into the test, eg VELIS, if there is no Genus One type prepared?

from ariston-remotethermo-home-assistant-v3.

VoyteckPL avatar VoyteckPL commented on September 23, 2024 1

He did this for my request. It is a horrow but it works 🙂

from ariston-remotethermo-home-assistant-v3.

Jedrek2021 avatar Jedrek2021 commented on September 23, 2024 1

Genus One 24 and Alteas One 24 are similar products. I took a quick look at the user manual and it seems that the timer programming for domestic hot water and home heating and the comfort temperature and economy temperature is determined by the controller used. For me, it's Sensys and what do you use?
Or maybe you will take a look at how it reads the Chomupashchuk v2 integration because it works great there and maybe the answer will be how to read it

ch_dhw_t

from ariston-remotethermo-home-assistant-v3.

Jedrek2021 avatar Jedrek2021 commented on September 23, 2024 1

Add central heating flow temperature - I checked on my Genus one + Sensys - it's ok

from ariston-remotethermo-home-assistant-v3.

VoyteckPL avatar VoyteckPL commented on September 23, 2024

+1

from ariston-remotethermo-home-assistant-v3.

bigstation17 avatar bigstation17 commented on September 23, 2024

+100 :)

from ariston-remotethermo-home-assistant-v3.

rikpal865 avatar rikpal865 commented on September 23, 2024

+1000 :)

from ariston-remotethermo-home-assistant-v3.

fustom avatar fustom commented on September 23, 2024

Sorry everyone, currently I am very busy. The second child is about to arrive, a full time job (as a developer), we are moving to countryside. But as far as I know Chomupashchuk fixed his integration.

from ariston-remotethermo-home-assistant-v3.

fustom avatar fustom commented on September 23, 2024

Hi @Jedrek2021
I fixed 1,2,6. But unfortunately I don't have genus one device. Can you confirm that they are working properly?

from ariston-remotethermo-home-assistant-v3.

Jedrek2021 avatar Jedrek2021 commented on September 23, 2024

I am still using Chomupashchuk's v2 integration but I was curious to check the changes in your integration because it is simple and fast. Integration does not recognize Genus One System but shows that it is GALEVO.

  • (1) There is no outside temperature - fixed
  • (2) There is no current DHW temperature (DHW storeage probe temperature) - fixed.
  • (6) Maybe it would be possible to read the current power (wish ....) - I do not see the parameter, there is energy,
    1 and 2 fixed - good job, thank you
    Problem no. 3 and 4 can be improved too? 3 seems to be more important. I enclose the view of the card.
    fixed_1_2

from ariston-remotethermo-home-assistant-v3.

fustom avatar fustom commented on September 23, 2024

Thank you for the response!

Integration does not recognize Genus One System but shows that it is GALEVO.

Yeah I know. This is the system type according to Ariston's website. There is no endpoint to get the real model. (At least I could not find it.) The official mobile app does not show that either.

(6) Maybe it would be possible to read the current power (wish ....) - I do not see the parameter, there is energy,

You're right I just realized you asked for current power consumption. The total energy sensor shows the last 1 hour (I am not 100% sure) consumption.

(3) When working TimeBase with a hamonogram, you cannot see any changes in the set temperature (its only the comfortable temperature - DHW comfort temperature and no change, no DHW economy temperature)

I don't know where to find DHW economy temperature. My Alteas One 24 only has "ZoneEconomyTemp" for CH thermostat. I need a temporary access to a Genus device to check is there any extra property for that.

(4) There is no information about is ch_flame or dhw_flame

This is probably the same problem. I only have "IsFlameOn" no matter DHW or CH.

from ariston-remotethermo-home-assistant-v3.

Jedrek2021 avatar Jedrek2021 commented on September 23, 2024

Determining whether we are heating the water or the house is very complicated. Now I understand why Chomupashchuk_v2 ch_flame and dhw_flame sometimes don't match my regulator in Genus One. It seems that setting ch_flame and dhw_flame separately is a waste of time, not a priority. What do others think about it?

from ariston-remotethermo-home-assistant-v3.

esistgut avatar esistgut commented on September 23, 2024

does anyone know if this has the equivalent of "mode" and "ch_mode" of chomupashchuk/ariston-remotethermo-home-assistant-v2 ? I can't really seem to set anything

from ariston-remotethermo-home-assistant-v3.

Jedrek2021 avatar Jedrek2021 commented on September 23, 2024

In the chomupashchuk/ariston-remotethermo-home-assistant-v2 integration, that is, it is like in the attached picture, is that what you are asking @esistgut
Ariston_mode

from ariston-remotethermo-home-assistant-v3.

esistgut avatar esistgut commented on September 23, 2024

@Jedrek2021 I know, that's way I asked for it in this integration. The v2 version has some kind of CPU usage bug chomupashchuk/ariston-remotethermo-home-assistant-v2#41 that is a deal breaker for people like me running it on a RaspberryPi.

from ariston-remotethermo-home-assistant-v3.

fustom avatar fustom commented on September 23, 2024

@esistgut There is no equivalent for that. I can add it if you still need it.

from ariston-remotethermo-home-assistant-v3.

bos-mary avatar bos-mary commented on September 23, 2024

Ok I checked ch_flame and dhw_flame on Chomupashchuk's v2 integration. That's actually a lie. https://github.com/chomupashchuk/ariston-remotethermo-home-assistant-v2/blob/af5b4f4d3b47901faa0c0ae0bd339e8a4f9364bc/custom_components/ariston/ariston.py#L1008

        if sensor == self._PARAM_DHW_FLAME:
            value = None
            try:
                increase_dhw_temp = None
                new_value = self._ariston_sensors[self._PARAM_DHW_STORAGE_TEMPERATURE][self._VALUE]
                if new_value:
                    increase_dhw_temp = False
                    if self._last_dhw_storage_temp is not None and \
                        self._last_dhw_storage_temp > 0 and \
                        new_value > 0:
                        if new_value > self._last_dhw_storage_temp:
                            increase_dhw_temp = True
                    elif self._last_dhw_storage_temp is not None and\
                        new_value > 0:
                        self._last_dhw_storage_temp = new_value
            except Exception:
                increase_dhw_temp = None
            ch_flame = None
            for zone in self._zones:
                ch_flame_zone = self._ariston_sensors[self._zone_sensor_name(self._PARAM_FLAME, zone)][self._VALUE]
                if ch_flame_zone in self._OFF_ON_TEXT:
                    if ch_flame is None or ch_flame == self._OFF:
                        ch_flame = ch_flame_zone
            if self._ariston_sensors[self._PARAM_FLAME][self._VALUE] in self._OFF_ON_TEXT and ch_flame in self._OFF_ON_TEXT:
                if self._ariston_sensors[self._PARAM_FLAME][self._VALUE] == self._OFF:
                    value = self._OFF
                elif ch_flame == self._OFF:
                    value = self._ON
                else:
                    # Unknown state of DHW
                    value = increase_dhw_temp
        return value

It is a horror, but as far as I understand: For dhw_flame he checks the zones for heat requests.

  1. No zone with heat request. Checks the flame flag.
    a) The flame is true => Returns dhw_flame 'ON'
    b) The flame is false => Returns dhw_flame 'OFF'
  2. There is a zone with heat request. He check is the DHW storage temperature higher then the last stored temperature.
    a) The DHW storage temperature higher => Returns dhw_flame True
    b) The DHW storage temperature not higher => Returns dhw_flame False
    c) Could not get DHW storage temperature => Returns dhw_flame None

For ch_flame he just returns the heat request for the zone. I can do the same. But I really do not want to do that.

In Genus One+ wifi manual there is parameter 824. If 824 = 1, the boiler is in CH mode, if 824=0 is in DHW mode. If you could read this parameter you could use it to create Flame_on_dhw and flame_on_ch.

Also please add 828 parameter to read the Instant Power of boiler.

from ariston-remotethermo-home-assistant-v3.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.