Skip to content

Add options to set battery capacity and range settings when new#420

Open
brunovale91 wants to merge 1 commit into
tobiasehlert:mainfrom
brunovale91:main
Open

Add options to set battery capacity and range settings when new#420
brunovale91 wants to merge 1 commit into
tobiasehlert:mainfrom
brunovale91:main

Conversation

@brunovale91

Copy link
Copy Markdown

This PR adds env variables to configure initial battery capacity and range. This will allow for proper battery health stats even when tesla mate does not have the full dataset since the purchase of the vehicle.

@LukStankovic

Copy link
Copy Markdown
Contributor

Hi @brunovale91, thank you 🙏🏻 I completely missed that when I created that endpoint. Maybe it would be enough to pass only two variables like it is in Grafana?

@brunovale91

Copy link
Copy Markdown
Author

Hi @brunovale91, thank you 🙏🏻 I completely missed that when I created that endpoint. Maybe it would be enough to pass only two variables like it is in Grafana?

Update to only use one variable for max range. Thank you for the API, great work!

@tobiasehlert

Copy link
Copy Markdown
Owner

Hmm.. well it would be working with Ken car, but what about those having multiple ones? Is it stored somewhere in some table from Grafana so we can access that?

@brunovale91

Copy link
Copy Markdown
Author

Good point, I don’t think accessing grafana database is a good option since the structure can change with grafana updates. I would say the env variables can be improved to take a list of tuples with car_id:value or tesla mate can be improved to support this car configuration (probably best option but the one that’s more difficult to do).

@tobiasehlert

Copy link
Copy Markdown
Owner

I think the best way would be to have this added in TeslaMate. There are car-specific settings and we should probably be able to extend those with this parameters as well, which in its turn can be consumed by Grafana or us, instead of this being something stored in Grafana directly.

@brunovale91, do you want to make a feature request on the TeslaMate repository? :)

@brunovale91

Copy link
Copy Markdown
Author

created a discussion for this: teslamate-org/teslamate#5087. Meanwhile going to change this PR to support a list of tupples.

@sonarqubecloud

sonarqubecloud Bot commented Jan 2, 2026

Copy link
Copy Markdown

@tobiasehlert

Copy link
Copy Markdown
Owner

Hello @brunovale91,

What you think about adding such a feature in TeslaMate application instead and then include that added information in our endpoints later on? I saw your discussion (teslamate-org/teslamate#5087) and also another issue (teslamate-org/teslamate#5039) that are just about that issue.

Regards,
Tobias

@brunovale91

Copy link
Copy Markdown
Author

@LukStankovic added a feature to HedgieMate which solves my specific case. I don't think it is ideal to have this on every client so I would still think it is better to have this feature implemented on the tesla mate application.

The solution on this PR also works but requires technical knowledge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

question Further information is requested

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants