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

"Range Added" has incorrect units on the Charges dashboard #4348

Closed
JakobLichterfeld opened this issue Nov 3, 2024 · 14 comments · Fixed by #4349
Closed

"Range Added" has incorrect units on the Charges dashboard #4348

JakobLichterfeld opened this issue Nov 3, 2024 · 14 comments · Fixed by #4349
Labels
area:dashboard Related to a Grafana dashboard kind:bug Something isn't working

Comments

@JakobLichterfeld
Copy link
Collaborator

JakobLichterfeld commented Nov 3, 2024

Hello, I'm a new user in this exact issue is something that bothered me enough to come here and search for answers. Was this issue corrected yet? I'm see the same behavior as op.

In the "range added" column, I would expect to see the number of miles that I gained during the charging session. Instead, it shows me how fast the charging occurred. This is super misleading because if I charged for 28 min and gained 20 miles of range, it displays 45 mph. If I was charged for exactly one hour, then that value would be accurate but I'd argue that most people are not charging for exactly one hour, and would rather see the number of miles worth of change during the amount of time they were plugged in and not for one hour.

Can this please be corrected or can there be an option to display the actual range added?

Originally posted by @1920x1200 in #4087 (comment)

@JakobLichterfeld
Copy link
Collaborator Author

@1920x1200 welcome to TeslaMate, nice to have you.

Please do not report an issue in a closed issue, it was only by accident I had notifications on for the closed issue.

Regarding the issue. @DrMichael solved it by renaming it to Range added / h, but @jheredianet accidentally reverted it when adding the Summary in #4144.

@swiffer What's your opinion on this? I personally do not see any benefit in Range added / h at all, but some do and would like to add it to MQTT as well #4130

@JakobLichterfeld JakobLichterfeld added kind:bug Something isn't working area:dashboard Related to a Grafana dashboard labels Nov 3, 2024
@JakobLichterfeld JakobLichterfeld changed the title "Range Added" is incorrect, and has incorrect units on the "Charges" dashboard "Range Added" has incorrect units on the "Charges" dashboard Nov 3, 2024
@JakobLichterfeld JakobLichterfeld changed the title "Range Added" has incorrect units on the "Charges" dashboard "Range Added" has incorrect units on the Charges dashboard Nov 3, 2024
@swiffer
Copy link
Contributor

swiffer commented Nov 3, 2024

I think it's the unit Tesla is showing during Super Charging as well on screen, don't they. I would therefore say let's keep it like that.

see here, down on bottom of video, KPI in the middle: https://www.tesla.com/fr_ch/blog/introducing-v3-supercharging

At this rate, a Model 3 Long Range operating at peak efficiency can recover up to 75 miles of charge in 5 minutes and charge at rates of up to 1,000 miles per hour.

So therefore, rename the column to "Charge Rate (avg)" instead of "Range added" ?

@JakobLichterfeld
Copy link
Collaborator Author

I think it's the unit Tesla is showing during Super Charging as well on screen, don't they. I would therefore say let's keep it like that.

Yeah it is.

So therefore, rename the column to "Charge Rate (avg)" instead of "Range added" ?

I do like it. A better name than range added / h

@swiffer
Copy link
Contributor

swiffer commented Nov 3, 2024

Ok, PR open in a sec ;)

@swiffer
Copy link
Contributor

swiffer commented Nov 3, 2024

I wonder if we should replace Driven (km) and Odometer (km) by Range added instead. Are those 2 really helping here?

@JakobLichterfeld
Copy link
Collaborator Author

JakobLichterfeld commented Nov 3, 2024

I wonder if we should replace Driven (km) and Odometer (km) by Range added instead. Are those 2 really helping here?

Odometer was recently added due to community request: #4109 (reply in thread)

I personally do not care about range added as I use added kWh

@swiffer
Copy link
Contributor

swiffer commented Nov 3, 2024

still voting to replace Driven with Range added then, Driven is hard to grasp as it is not really related to the charge

@DrMichael
Copy link
Collaborator

So therefore, rename the column to "Charge Rate (avg)" instead of "Range added" ?

Appears clearer. We could also name the column just "km /hr" and "mi / hr". (When looking at the screenshot.

@DrMichael
Copy link
Collaborator

still voting to replace Driven with Range added then, Driven is hard to grasp as it is not really related to the charge

Aren't that different things? Driven means distance driven since last charge, right?

I think, Odometer is fine. But yes, Driven and Odometer cover the same issue.

@JakobLichterfeld
Copy link
Collaborator Author

Agree, what do you think @DrMichael ?

@swiffer
Copy link
Contributor

swiffer commented Nov 3, 2024

still voting to replace Driven with Range added then, Driven is hard to grasp as it is not really related to the charge

Aren't that different things? Driven means distance driven since last charge, right?

Yes, different things, I myself finding driven not really helpful in context of charges, that's why I wanted to replace it.

I think, Odometer is fine. But yes, Driven and Odometer cover the same issue.

@1920x1200
Copy link

If we're going to keep the speed of charging (in MPH), is there a way to add a column for the number of miles added during a charging session? Personally, I would find that more useful.

@swiffer
Copy link
Contributor

swiffer commented Nov 3, 2024

That's what I suggested - will open a PR later

@1920x1200
Copy link

Thank you for fixing this in the latest update!

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

Successfully merging a pull request may close this issue.

4 participants