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

minor/optical feature request: use kWh (not Wh) in YealdDay (adjust api/mqtt) #2260

Open
4 tasks done
harvey637 opened this issue Sep 8, 2024 · 2 comments
Open
4 tasks done
Labels
bug Something isn't working

Comments

@harvey637
Copy link

What happened?

in the general tab the yealdDay is displayed in different units (Wh) than the YealdTotal (kWh).
As both values will jump over 1 kWh easily it is more easy to read both values in the same unit. The mixture of separators (omma or dot) might even confuse also.
The single module might stay as it is, as yieldday might no be so high in kWh.
Or think about autoscale if value exceeds 1 kWh .... (but not in the api/mqtt)

To Reproduce Bug

see in the gui

Expected Behavior

use easy comparable units scales ....

and very minor: the textual explanation of the "yealdtotal" is wrongly "Energy converted to AC since reset watt hours" but it is in "kWh" :-) yealdday and yealdtotal both in "kWh" ????

Install Method

Pre-Compiled binary from GitHub

What git-hash/version of OpenDTU?

24.8.5

Relevant log/trace output

No response

Anything else?

No response

Please confirm the following

  • I believe this issue is a bug that affects all users of OpenDTU, not something specific to my installation.
  • I have already searched for relevant existing issues and discussions before opening this report.
  • I have updated the title field above with a concise description.
  • I have double checked that my inverter does not contain a W in the model name (like HMS-xxxW) as they are not supported
@harvey637 harvey637 added the bug Something isn't working label Sep 8, 2024
@creaimmo
Copy link

creaimmo commented Dec 21, 2024

Hello,

I notice a display bug on the Total Yield value. Either a (.) or (,) is missing but the value expressed in kWh is wrong because this element is missing.

My real value : 2075,784 kWh

share_272826804199538502

@tbnobody
Copy link
Owner

tbnobody commented Dec 21, 2024

@creaimmo this format is generated by your browser.

at least with the latest versions of chrome and firefox it looks like this:
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants