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

HASS.IO v0.97.1 Lovelace Card not displaying correct KNX Operation Modes #26624

Closed
G8YTZ opened this issue Aug 10, 2019 · 78 comments · Fixed by #39219
Closed

HASS.IO v0.97.1 Lovelace Card not displaying correct KNX Operation Modes #26624

G8YTZ opened this issue Aug 10, 2019 · 78 comments · Fixed by #39219

Comments

@G8YTZ
Copy link

G8YTZ commented Aug 10, 2019

In the States UI thermostats appear with the ability to select one of the 4 KNX Operation Modes. With 0.97 the Lovelace Card only displays 3 Operation modes and these do not work to change the Operation mode. The picture shows the Graphic that is incorrect, but the "drill-down" is ok.
Screenshot 2019-08-10 at 15 08 49

@frenck frenck transferred this issue from home-assistant/home-assistant.io Aug 12, 2019
@probot-home-assistant
Copy link

Hey there @Julius2342, mind taking a look at this issue as its been labeled with a integration (knx) you are listed as a codeowner for? Thanks!

@G8YTZ
Copy link
Author

G8YTZ commented Aug 12, 2019

0.97.2 has the same issue

@elupus
Copy link
Contributor

elupus commented Aug 13, 2019

This look like it's correct. HA only display modes in GUI. Presets are only in the drop-down.

So graphics will only show: off, auto, heat, cool.

@G8YTZ
Copy link
Author

G8YTZ commented Aug 13, 2019 via email

@elupus
Copy link
Contributor

elupus commented Aug 13, 2019

Climate changed in 0.96: https://www.home-assistant.io/blog/2019/07/17/release-96/

@G8YTZ
Copy link
Author

G8YTZ commented Aug 15, 2019

HI, this issue is far from closed. The issue relates specifically to the Lovelace Climate Card not displaying the 4 KNX operation modes from 0.96.x, it is not fixed in 0.97.x either. Look at my config above, it is to the book

To summarise: There are no issues using the STATES-UI, neither are there any issue using the LOVELACE UI using a simple thermostat, for example:

- type: entities title: Lounge entities: - entity: climate.lounge_south_climate icon: mdi:radiator

But using the Thermostat Card is where it all goes wrong. MY configuration in UI-LOVELACE.yaml is as follows:

` - type: vertical-stack # column 2
cards:

      - type: horizontal-stack
        cards:
          - type: thermostat
            entity: climate.master_suite_climate
            name: Master Bedroom`

So something has gone badly wrong with the Thermostat Card from 0.96

@elupus
Copy link
Contributor

elupus commented Aug 15, 2019

It is not that broken. There are not that many modes to display. "Comfort" is no longer a mode for climate. It's a profile. Lovelace card will only display valid modes, not profiles.

Yes it's reduction in function. Motivation is to keep modes in line with Alexa and Google Assistant.

@G8YTZ
Copy link
Author

G8YTZ commented Aug 15, 2019

That just cannot be true, as I say it is only the Thermostat Card in Lovelace that is broken (the most important user interface) The fundamental STATES-UI work perfectly ok, so how can Lovelace be anything to do with Alexa?

OK so how should profiles be configured? The documentation is either incorrect or missing something really important. If it is intentional then I believe a re-think should be in order, grief even the documentation shows the Lovelace-UI example that worked for me a couple of weeks ago.

@elupus
Copy link
Contributor

elupus commented Aug 17, 2019

Can you link the documentation that you mention?

@elupus
Copy link
Contributor

elupus commented Aug 17, 2019

I don't fullt understand why you say graphics is wrong. It shows icons for selecting the mode. Ie off, auto, heat, cool.

There is no graphics for comfort, away, sleep, eco. Those you need to drill down into. Same goes for small also large Lovelace card, no difference there.

@G8YTZ
Copy link
Author

G8YTZ commented Aug 17, 2019

Sure, the KNX Page shows the configuration.yaml entries, mine are identical (apart from the addresses of course) and worked perfectly. In the UI- Lovelace .yaml the configuration for a thermostat card, until 0.95.4 everything worked well - you saw the 4 modes and as you switched modes the ocular changed, so did the set point and as you switched modes this would be reflected in the graphics. Now there are only 3 modes shown and none of them work and the colours don't change.

KNX Climate:
https://www.home-assistant.io/components/climate.knx/

Lovelace
https://www.home-assistant.io/lovelace/thermostat/ - Note the 4 KNX modes!

KNX is the worldwide standard for building control - it's a professional solution for systems integrators:

The issue seems to be with the Thermostat Card as the STATES-UI is just fine. Many thanks for your help in anticipation.

Screenshot 2019-08-17 at 16 48 41

@G8YTZ
Copy link
Author

G8YTZ commented Aug 17, 2019

I was going to say, we could do a WebEX if you would like to see the problem first hand.

@selfisch
Copy link

selfisch commented Aug 23, 2019

Hey,
I dont exactly understand your initial Post

The picture shows the Graphic that is incorrect, but the "drill-down" is ok.
But in your screenshot, everything seems fine.

For me nothing changed. Here an example of my configuration:

- platform: knx
  name: Wohnzimmer
  min_temp: 17.0
  max_temp: 23.0
  temperature_address: '1/1/11'
  target_temperature_state_address: '1/1/12'
  setpoint_shift_address: '1/1/13'
  operation_mode_address: '1/1/19'
Lovelace UI config:
  - type: entities
    title: Wohnzimmer
    show_header_toggle: false
    entities:
      - entity: climate.wohnzimmer

and a screenshot of the UI:
image

and last but not least, a screenshot of knx debug log:
image

So if I switch to one of the 4 modes, everything is fine.
I am running HA0.97.2

@G8YTZ
Copy link
Author

G8YTZ commented Aug 23, 2019

In the Lovelace UI Thermostat card only 3 modes are shown, but on the STATES UI all 4 are shown and this work properly. Also the mode buttons no longer work on the Lovelace UI and the state colours are not shown either.

The Lovelace UI Thermostat card has been useless since v 0.96, yet it worked perfectly on previous versions.

The key issue is with the Lovelace Thermostat Card. The picture here shows only three operation modes available when there should be 4 like there always was before.

Screenshot 2019-08-23 at 12 55 51

@MartinHjelmare
Copy link
Member

I think you will get better visibility if we transfer this issue to the frontend repo, since it sounds like it's a frontend issue. Although it sounds like there has already been an answer to the change of behavior.

https://github.com/home-assistant/home-assistant/issues/25904#issuecomment-521810026

@MartinHjelmare MartinHjelmare transferred this issue from home-assistant/core Aug 23, 2019
@G8YTZ
Copy link
Author

G8YTZ commented Aug 23, 2019

I did see that comment about reduction in functionality, but that cannot be true, frankly if this does not work I'm wasting my time going with Home Assistant.

KNX is the worldwide BMS standard with massive adoption, Google and the like are just toys.

@balloob
Copy link
Member

balloob commented Aug 23, 2019

No functionality was reduced, it's just accessible as a preset instead of an hvac mode.

@G8YTZ
Copy link
Author

G8YTZ commented Aug 23, 2019

The whole Lovelace Thermostat interface is now basically useless, but the documentation does not say this. How is the current operation mode displayed? How is it usable from a finger on an iPhone or on a car dashboard or a touchscreen phone like the Gigaset Maxwell 10?
This was what was so brilliant - easy to use and not it does not work with the leading BMS system. That cannot be right.

@selfisch
Copy link

In the Lovelace UI Thermostat card only 3 modes are shown, but on the STATES UI all 4 are shown and this work properly. Also the mode buttons no longer work on the Lovelace UI and the state colours are not shown either.

The Lovelace UI Thermostat card has been useless since v 0.96, yet it worked perfectly on previous versions.

The key issue is with the Lovelace Thermostat Card. The picture here shows only three operation modes available when there should be 4 like there always was before.

Screenshot 2019-08-23 at 12 55 51

Now I got the point, sorry for that!
You are absolutely right.
I am currently not using that kind of card in lovelace ui, but hopefully those modes wont be taken away completely from the knx climate integration :-)

@G8YTZ
Copy link
Author

G8YTZ commented Aug 23, 2019

Cool! here is what the thermostat page looks like (prior to 0.96) on my Tesla Screen/ Nabu Casa, you can see the thermostats have all modes and can easily be operated by a finger. My plan is to buy the Gigaset Maxwell 10S VOIP phone for my Asterisk system and use the Android OS on the phone to install the HA App and use it to control functions for that room, situated next to the sofa on the coffee table. I think that would be great use for HA as a room controller.

IMG_2041

@G8YTZ
Copy link
Author

G8YTZ commented Aug 23, 2019

A video also....

https://youtu.be/10nqj6f0uwc

@Mariusthvdb
Copy link
Contributor

wow, looking mightily impressive. But please don't tell us you have 12 thermostats ..? (sorry if this diverges from the OP)

@G8YTZ
Copy link
Author

G8YTZ commented Aug 23, 2019

The house is KNX, only diverges for things like Yale Conexis integration (via Smartthings Hub) external web sensors, HIK Vision Cameras, Cars etc. I took a KNX certification course 10 years ago when planning the house self-build.

The heating consists of a Mitsubishi Ecodan air-water Heat Pump, Theben HMT12 actuator that controls 11 underfloor zones and then there are 4 electric towels radiators in the bathrooms, so there are actually 15 zones, though several are dual heat zones.

The thing I like about KNX is that it's and open protocol and a "walled garden" - one configuration tool for all manufacturers and 100% compatibility between manufacturers is gauranteed. The design life of the kit is about 25 years, no software updates required. It just works and does not become obsolete.

Next integration challenge is the Resol solar thermal controller, might do the PV too, but right now I use a Zennio KEX KNX current sensor to measure power.

@G8YTZ
Copy link
Author

G8YTZ commented Sep 11, 2019

Hi, is anyone looking at this issue since 0.96? It has completely broken the KNX thermostat integration. Someone has not thought these changes through, a major backward step that writes off HASS for use with KNX which is the largest professional BMS out there.

@iantrich
Copy link
Member

The card shows modes, not presets. You can access the more-info dialog which shows the presets by clicking the top right icon. I don't consider this issue based on your description. The modes of the integration changed, not our display.

@G8YTZ
Copy link
Author

G8YTZ commented Sep 11, 2019

Yes the Operation Modes don't work anymore. It has completely destroyed the UI with Lovelace how can the card show and control the operation modes correctly again? There must be a fix. Whatever had been done has ruined the interface. Check out how the colours used to change with mode too, not any more.

@iantrich
Copy link
Member

iantrich commented Sep 11, 2019

I see. People seemed to be focusing on the number of modes rather than the failure of the buttons and is what I picked up on as well.

So to be clear, all the mode buttons on the card no longer work for you but selecting from the drop-down does work?

@G8YTZ
Copy link
Author

G8YTZ commented Sep 11, 2019

Yes, that's correct, also you see only 3 operation mode buttons and before there were 4. The picture on the Tesla Dash vs the other (post 0.96) pictures I took. The Lovelace card lends itself very well to a touchscreen, the drop-down does not.

@G8YTZ
Copy link
Author

G8YTZ commented Mar 3, 2020 via email

@elupus
Copy link
Contributor

elupus commented Mar 3, 2020

That is because it's not a bug. It may be a lack of the feature to set presets directly from GUI. But it's not a bug.

@G8YTZ
Copy link
Author

G8YTZ commented Mar 3, 2020 via email

@elupus
Copy link
Contributor

elupus commented Mar 4, 2020

My home run knx.. I know what it is. But pressing the more info button to get to profiles is not that huge a problem.
Also, it should not be too hard to install the custom GUI https://github.com/nervetattoo/simple-thermostat to be able to control that.

@G8YTZ
Copy link
Author

G8YTZ commented Mar 4, 2020 via email

@pvizeli
Copy link
Member

pvizeli commented Mar 4, 2020

@Julius2342 was the maintainer and is not active around that anymore. So you need to wait until a new developer or exists developer buys a KNX and will fix that. Since KNX is not just cheap, I don't think that will happen soon.

@cyberjunky
Copy link
Contributor

@pvizeli, too bad @Julius2342 doesn't maintain the KNX integration anymore, I use KNX too.
I also created the toon_climate custom component, and had to fiddle with the code too to get it working after 0.96. Alas only mode settings are available in lovelace main thermostat card, you need the three dot menu to reach presets. Or use simple-thermostats like suggested above and option:

control:
          - preset

But I see the modes are also not displayed correctly (at least on/off heating should be there?)
Will have a look for what it's worth.

@cyberjunky
Copy link
Contributor

cyberjunky commented Mar 10, 2020

Actually, simple-thermostat,js works like you want I guess.
Below your example. above the simple-thermostat like this:

      - entity: climate.wohnzimmer
        control:
          - preset
        type: 'custom:simple-thermostat'

Screenshot from 2020-03-10 14-35-46

Never mind the bogus temp settings, my thermostat is working on other ga's.

@cyberjunky
Copy link
Contributor

Frontend devs should add a preset control flag and icons to the thermostat card as well.

@Julius2342
Copy link
Contributor

Julius2342 commented Mar 10, 2020

Two kids at home don't scale. @pvizeli : Is there anybody on hass team, who can help us fixing the Climate integration, mainly how to convert the new settings into KNX climate modes?

Or more specific: tell me what was changed on HASS side which broke the climate modes on KNX. From what i see, the core of the problem is, that HASS introduced new climate modes - and oriented on modern devices like NEST. These modes are incompatible with KNX Modes and all mitigations yet did not work. Or made it even worse.

@cyberjunky
Copy link
Contributor

What I noticed just now is that KNX climate doens't have hvac_action() implemented, which should return something like this (pseudo code)

    @property
    def hvac_action(self) -> Optional[str]:
        """Return the current running hvac operation."""
        if self._heating:
            return CURRENT_HVAC_HEAT
        if self._off:
            return CURRENT_HVAC_OFF
        else:
            return CURRENT_HVAC_IDLE

Will have a look

@cyberjunky
Copy link
Contributor

Looking at config page and code, it all seems to work as designed (to new lovelace/climate standard)

climate presets:
All presets are available (but only via three dot menu, this is by design, unless changed)
Can be made more user friendly using simple-thermostat and control preset setting. or by change in design of lovelace thermostat. (frontend devs)

climate operation modes:
In my first attempt, and in some screenshots above only the auto mode icon is visible.
But if you specify on_off_address and on_off_state_address the device is seen as on_off capable and also shows the heat and off icon.

If not correct, please specify clearly what is missing/wrong, the long thread above is difficult to read through.

@G8YTZ
Copy link
Author

G8YTZ commented Apr 29, 2020

I still don't understand why this Lovelace bug has not been fixed. Why can't we just roll back to as t was in 0.96 when it worked perfectly? Nobody asked for this change.

What people would like is an integrated scheduler (including thermostats where the operation mode change is available in the scheduler and with a decent user interface for the wife to use.

@elupus
Copy link
Contributor

elupus commented Apr 29, 2020

Because it's not a bug

@G8YTZ
Copy link
Author

G8YTZ commented Apr 29, 2020

Everything worked perfectly before 0.97 and then it did not - so its a bug or fault my book. Faulty design perhaps. The designer failed to understand the consequences for a proper professional smart building solution like KNX and the power of knx operation modes in energy management.

@elupus
Copy link
Contributor

elupus commented Apr 29, 2020

You can then easily afford the time to write a custom Lovelace card to support your need for direct touch access to profiles. They work perfectly fine from the sub menu.

@G8YTZ
Copy link
Author

G8YTZ commented Apr 29, 2020 via email

@stale
Copy link

stale bot commented Jul 29, 2020

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates.
Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍
This issue now has been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Jul 29, 2020
@G8YTZ
Copy link
Author

G8YTZ commented Jul 29, 2020 via email

@stale stale bot removed the stale label Jul 29, 2020
@marvin-w
Copy link
Contributor

marvin-w commented Aug 23, 2020

@G8YTZ You should have a read through home-assistant/architecture#22. This was the architectural design decision that forced us to implement a proper mapping to the available HomeAssistant HVACModes in the backend.

Please also read specifically this comment that makes it clear that we actively participated there: home-assistant/architecture#22 (comment)

From an architecture perspective this has been a tremendously huge change involving a lot of integrations and will not be rolled back (as it was a perfectly fine move from HA to streamline the climate entity).

Let me clarify this: The old lovelace card is not going to come back, you will always need to use the presets in order to do what you are trying to achieve.

Consider moving to the custom card mentioned by @cyberjunky if you can't afford to open the three dot menu.

I think there still is some weird issue with the duplicated HVAC_HEAT_MODE that should be fixed with the next release of xknx (if I recall correctly).

From my perspective the issue reported here is not an issue, but rather a result of a new architecture (and the related frontend changes).

Please also note one more thing: Due to the fact that XKNX supports literally all KNX devices for a specific type it's very hard to test if everything is working correctly since one usually doesn't own all needed devices from all vendors in all different variations in order to make a complete test.

@G8YTZ
Copy link
Author

G8YTZ commented Aug 23, 2020 via email

@marvin-w
Copy link
Contributor

I mean it still is compatible, you just have to click one more button or just use that custom card that was mentioned.

We know the integration isn't perfect, but you have to make compromises in order to support multiple smart home solutions. Please test it. There will not be any improvement to the current lovelace card in terms of usability for KNX as this would break the integration for other climate platforms.

@G8YTZ
Copy link
Author

G8YTZ commented Aug 23, 2020 via email

@farmio
Copy link
Contributor

farmio commented Aug 23, 2020

This should be fixed. The default climate havc_mode is now "heat" since #35154 so it isn't viewed as "off" (grey) anymore.

@marvin-w
Copy link
Contributor

You can always roll back to HA < 0.97 but this change will not come back. Feel free to use one of the alternatives provided or write your own custom lovelace card.

@G8YTZ
Copy link
Author

G8YTZ commented Aug 23, 2020 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.