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

Battery Health - LFP - Status: 500. Message: convert frame from rows error: pq: division by zero #3675

Closed
1 task done
henzay opened this issue Feb 14, 2024 · 4 comments · Fixed by #4007 or #4038
Closed
1 task done
Labels
area:grafana Related to Grafana undetermined Not sure if this is considered a real bug

Comments

@henzay
Copy link

henzay commented Feb 14, 2024

Is there an existing issue for this?

  • I have searched the existing issues

What happened?

I am receiving an error when I use the battery health LFP screen in the graph "Battery Capacity by Mileage"

Expected Behavior

Normally a red line will be displayed to show the batttery capacity degradation.
here it is not displayed.

Steps To Reproduce

Go to the Battery Health - LFP screen

Relevant log output

teslamate-1  | 2024-02-14 08:45:03.356 [info] GET /
teslamate-1  | 2024-02-14 08:45:03.361 [info] Sent 200 in 4ms
teslamate-1  | 2024-02-14 10:28:16.281 [info] Refreshing access token ...
teslamate-1  | 2024-02-14 10:28:16.761 [info] POST https://auth.tesla.com/oauth2/v3/token -> 200 (479.380 ms)
teslamate-1  | 2024-02-14 10:28:16.764 [info] Scheduling token refresh in 6 h
teslamate-1  | 2024-02-14 13:47:21.412 [info] GET /
teslamate-1  | 2024-02-14 13:47:21.419 [info] Sent 200 in 6ms
teslamate-1  | 2024-02-14 14:06:42.560 [info] GET /
teslamate-1  | 2024-02-14 14:06:42.566 [info] Sent 200 in 6ms

Screenshots

Screenshot 2024-02-14 at 15 08 37

Additional data

No response

Type of installation

Docker

Version

v1.28.3

@JakobLichterfeld
Copy link
Collaborator

JakobLichterfeld commented Feb 15, 2024

already solved by #3593, will be included in next release

@JakobLichterfeld
Copy link
Collaborator

Sorry, my bad, #3593 is already included in 1.28.3

@jheredianet can you please take a look into this?

@JakobLichterfeld JakobLichterfeld added undetermined Not sure if this is considered a real bug area:grafana Related to Grafana labels Feb 17, 2024
@jheredianet
Copy link
Contributor

Sorry, my bad, #3593 is already included in 1.28.3

@jheredianet can you please take a look into this?

Hi, sorry, I can only speak on the original "Battery Health" dashboard as the creator of it.
As I've explained in other post, IMHO, there is no need to have another battery health dashboard for LFP batteries.

Somebody cloned the dashboard into LPF one, with "some adjustments" and his PR has been accepted as valid, but then it has not carried out the respective maintenance, since the original dashboard has been modified and adjusted to avoid calculation errors and null values and take care of different units of measurement.

So, @henzay I guess, the error "divided by 0" on the original "Battery Health" dashboard is not happening. If so, that's the dashboard you would need to use.

I've tried to remove the LFP dashboards in a PR, but it was denied. The best option is to implement this feature.

@JakobLichterfeld
Copy link
Collaborator

JakobLichterfeld commented Feb 18, 2024

Hi, sorry, I can only speak on the original "Battery Health" dashboard as the creator of it. As I've explained in other post, IMHO, there is no need to have another battery health dashboard for LFP batteries.

Thank you for jogging my memory.
We need to maintain both dashboards until #3438 is not solved. No single PR will be accepted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:grafana Related to Grafana undetermined Not sure if this is considered a real bug
Projects
None yet
3 participants