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

Provide cumulative sensors as well as the "today so far" counters #25

Closed
Kallb123 opened this issue Sep 13, 2021 · 5 comments · Fixed by #49
Closed

Provide cumulative sensors as well as the "today so far" counters #25

Kallb123 opened this issue Sep 13, 2021 · 5 comments · Fixed by #49
Labels
enhancement New feature or request

Comments

@Kallb123
Copy link

Is your feature request related to a problem? Please describe.
Using the current "Today" sensors within the energy dashboard results in flawed data at midnight. The dashboard expects ever-increasing data, like a meter reading. Whereas the sensors currently reset each day and provide a saw-tooth data set.

Describe the solution you'd like
Add an additional sensor to provide cumulative data that doesn't reset.

Describe alternatives you've considered
For now I'm likely to use NodeRed to detect the midnight reset and then keep track of incremental usage through the day, this can be added onto a total that keeps increasing.

Additional context
Screenshot_20210913-073008

@Kallb123 Kallb123 added the enhancement New feature or request label Sep 13, 2021
@ColinRobbins
Copy link
Contributor

Are you seeing the issue for both gas and electricity?
I've integrated both, but am not seeing the issue.
What version of Hass are you on?

@Kallb123
Copy link
Author

Kallb123 commented Oct 8, 2021

I'm currently on HASS 2021.9.7
Yes it's on both types of fuel. You can see that there's a reduced amount at midnight on electricity and you can see the massive negative spike on the gas.

The entities have been unavailable for a couple of weeks now and I've not bothered trying to fix it since the Bright app is only showing me days from yesterday.

@ColinRobbins
Copy link
Contributor

The entries unavailable problem is issue #14.
I will shortly be creating a pull request, as I have a fix running locally.

@ColinRobbins
Copy link
Contributor

I have removed the pull request for entries unavailable problem is issue #14. but fixed the issue in my fork.
The reason for fixing in a fork this is discussed here

@ColinRobbins
Copy link
Contributor

I now have the cumulative sensors implemented in my test system.
Need to let it run for a few days to make sure the Energy integration works OK with it.
If all goes well, I'll make a PR.

@ColinRobbins ColinRobbins mentioned this issue Oct 23, 2021
3 tasks
@HandyHat HandyHat linked a pull request Oct 24, 2021 that will close this issue
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants