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

Add to the attributes the next switch change. #71

Open
miki3421 opened this issue Jan 25, 2022 · 16 comments
Open

Add to the attributes the next switch change. #71

miki3421 opened this issue Jan 25, 2022 · 16 comments

Comments

@miki3421
Copy link

Is your feature request related to a problem? Please describe.
At the moment this value is present on the HA log and I need to keep open the terminal to read the values.

Describe the solution you'd like
Would be the best having this in an attribute to read directly from the an HA lovelace card.

Describe alternatives you've considered
Value already present in the log.

Additional context
this is an example:
Time until switch.gas_miki_office turns on for Smart Miki Office (climate.smart_miki_office): 2063 sec

Time_next_change: 2036 sec.

And thank to the developer for this amazing integration. Really it works so well.

@miki3421
Copy link
Author

miki3421 commented Feb 2, 2022

I've made this change on my own. Let me know if I can post here the updated version of the climate.py

@bigbread59
Copy link

@miki3421
Great!!
I am very interested in this feature!

@miki3421
Copy link
Author

miki3421 commented Feb 2, 2022

Here you can find the climate.py file that is creating time_to_off and time_to_on as attributes.
just overwrite the file that is in the directory /custom_components/smart_thermostat.
I suggest anyway to make a copy of the original file in case of problems.
climate.py.zip

@bigbread59
Copy link

Here you can find the climate.py file that is creating time_to_off and time_to_on as attributes. just overwrite the file that is in the directory /custom_components/smart_thermostat. I suggest anyway to make a copy of the original file in case of problems. climate.py.zip

Thanks, i'll try

@miki3421
Copy link
Author

miki3421 commented Feb 3, 2022

Here you can find the climate.py file that is creating time_to_off and time_to_on as attributes. just overwrite the file that is in the directory /custom_components/smart_thermostat. I suggest anyway to make a copy of the original file in case of problems. climate.py.zip

Thanks, i'll try

Let me know if is working or not.

@bigbread59
Copy link

@miki3421
@ScratMan

It works well.

It would be better to include this feature in the official version.

@dorinvlas
Copy link

Guys can you provide some screenshots with this implementation? It would be very interesting to see the actual visuals. Thanks

@miki3421
Copy link
Author

miki3421 commented Nov 8, 2022 via email

@dorinvlas
Copy link

I don't see the images... only some tags like [image: image.png]. Thanks

@miki3421
Copy link
Author

miki3421 commented Nov 8, 2022

Screenshot 2022-11-08 at 08 55 29

image

@dorinvlas
Copy link

WOW niceee! Thanks.

Why you did not convert the seconds in the to minutes? :)

@miki3421
Copy link
Author

miki3421 commented Nov 8, 2022

You can do it. I don't need actually, once is set properly it works very well.

@dorinvlas
Copy link

dorinvlas commented Nov 8, 2022 via email

@ScratMan
Copy link
Owner

ScratMan commented Nov 9, 2022

Hello, I'm sorry I didn't work on the thermostat for a while, arrival of baby #2 drastically reduced my free time. I hope to be able to make a new build before end of the year (as Christmas gift)

There is a need for a rework of the thermostat attributes because as is, every change in one of the attributes will trigger storing a new state in the database for the thermostat with all the attributes, even those that didn't change. It increases the size of the database and reduces the performance of HA. So I need to change the code to send the PID and debug attributes in some optional sensors that should be enabled or disabled by configuration script.

@T81
Copy link

T81 commented Dec 6, 2022

Screenshot 2022-11-08 at 08 55 29

image

Which card is the first one? It looks very informative

@mannebk79
Copy link

There is a need for a rework of the thermostat attributes because as is, every change in one of the attributes will trigger storing a new state in the database for the thermostat with all the attributes, even those that didn't change. It increases the size of the database and reduces the performance of HA.

I already see that after one week with two smart thermostats... it really slows the system down, and i do not use the stock HA database but MariaDB for the reason the stock db is so slow if you start to pump in data like an own weather station and other trigger intensive stuff.

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

No branches or pull requests

6 participants