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 Dates in event log in Live-View #1800

Open
martinstejskal opened this issue Mar 1, 2024 · 6 comments
Open

Add Dates in event log in Live-View #1800

martinstejskal opened this issue Mar 1, 2024 · 6 comments
Labels
enhancement New feature or request

Comments

@martinstejskal
Copy link

Is your feature request related to a problem? Please describe.

There are only timestamps in event log. It is hard to understand what event was raised on what day. In my setup I use batteries, so my inverter runs 24/7.

Describe the solution you'd like

Add the date additionally to the timestamp in event log.

Describe alternatives you've considered

Alternatively the date could be shown on mouseover as a tooltip.

Additional context

This feature request increases the meaningfulness of the event log, by adding needed information of the date of an event.

@martinstejskal martinstejskal added the enhancement New feature or request label Mar 1, 2024
@tbnobody
Copy link
Owner

tbnobody commented Mar 1, 2024

The eventlog is reset at each reboot of the inverter. Therefor you can 100% assume that it's only the current day.
On the other hand, in the event view os only shown what the inverter deliveres.

@phol
Copy link

phol commented Mar 1, 2024

I would be in favour of adding dates to the event log, as I'm using the websocket to log OpenDTU into a database. Adding the date would ensure that I can use a condition like "IF CURRENT DAY", so events aren't logged multiple times.

@tbnobody
Copy link
Owner

tbnobody commented Mar 1, 2024

But I have no chance to know on which day the event occoured. If you e.g. don't reset the inverter over the night, the inverter returns something like 37:00:00.... Now I know that the time is 37:00:00 - 24:00:00 = 1 day 13:00....
But I don't know which date was 1 day... because it depends on the startup date of the inverter (which is not known).

@phol
Copy link

phol commented Mar 1, 2024

Ok that makes sense, as it's the exact same issue I'm facing when trying to parse the websocket data and removing duplicate events.
Perhaps adding a separate date field, indicating on which date the OpenDTU received the event from the receiver would be an option? Or adding some logic to remove duplicates?

@tbnobody
Copy link
Owner

tbnobody commented Mar 1, 2024

The problem is here, that the inverter sends ALL events. I request the events and get a response with the last 15 events. The inverter itself remove some events which seem to be less important if an more then 15 events occour on one day.

@phol
Copy link

phol commented Mar 28, 2024

To (at least partially) solve this problem, I've tried to analyse it and have made a proposal / small feature request in #1875 .

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

No branches or pull requests

3 participants