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 drain on EV? #184

Closed
tuxedo0801 opened this issue Dec 22, 2021 · 12 comments
Closed

Battery drain on EV? #184

tuxedo0801 opened this issue Dec 22, 2021 · 12 comments

Comments

@tuxedo0801
Copy link

Hi there,

just wondering whether regular queries (every 5mins?) will drain my EVs 12V battery?!
Does anyone know?

@PierreLevres
Copy link
Contributor

PierreLevres commented Dec 22, 2021 via email

@tuxedo0801
Copy link
Author

Great. Then you should write this in big upper case letters into the readme, or?

@Hacksore
Copy link
Owner

The default value for refresh is false

https://github.com/Hacksore/bluelinky/blob/develop/src/constants.ts#L26

@tuxedo0801
Copy link
Author

And how this will stop people from doing regular refresh by an endless loop? And how will they know that regular refreshes drain their battery?

This is a serious issue. Got a new car, used the library via nodered and queried status every 5min, because I thought it works like Renault: always ask the Cloud, not the car itself.
Result: struggled two days with broken battery and went to the garage without any finding. But finally I remembered my API queries ... And now I'm here.

I'm pretty sure that it makes a lot of sense to tell the people via Readme file that making regular queries can drain their car battery until the car will reject any interaction, even unlocking by remote key.

@Hacksore
Copy link
Owner

If you are doing endless refresh queries what else would you expect?

I understand that a message in the readme would be beneficial, submit a PR.

@PierreLevres
Copy link
Contributor

If you are doing endless refresh queries what else would you expect?

I understand that a message in the readme would be beneficial, submit a PR.

I did...

@tuxedo0801
Copy link
Author

If you are doing endless refresh queries what else would you expect?

How do you think people know that every query is sent to their car? I expected to query the last status the car has sent to the cloud from the cloud. I never expected that every query is sent to the car. I expected some kind of indirection. But this is more like a direct connection, with the cloud as a proxy.
That's why I find it so confusing. And that's why I expect to get warned by readme

@PierreLevres
Copy link
Contributor

You might wanna try pressing the refresh button in you app every 5 minutes. And when your battery is drained send a formal complaint to Kia/Hyundai that the app didn't give you any warning? I wonder if they would change the readme file.

And instead of being irritated that the readme isn't up to your standards you could also be grateful that a group of volunteers is spending their precious spare time building something for the community.
You could also try to contribute by submitting a PR for the things you like to be changed.

@Hacksore
Copy link
Owner

Merged #185 which should give a warning in the readme. Feel free to continue the discussion but I'm closing this issue.

@cr0ste
Copy link

cr0ste commented Feb 9, 2024

Despite of the adapted ReadMe, there still seems to be some confusion about it, as it's not very clear to non-insiders what "refresh" means.

What I understand from this discussion is that the car battery is only drained when using "refresh: true".
In several communities poeple are discussing about battery drain of bluelinky and interprete the new ReadMe as "You should not call bluelinky too often" (not knowing about refresh true/false toggle).

Can one of you please state if my assumption is right that no drain should happen with refresh=false?
If so, I would probably create another PR to make it more clear ;-)

@PierreLevres
Copy link
Contributor

PierreLevres commented Feb 10, 2024 via email

@cr0ste
Copy link

cr0ste commented Apr 20, 2024

@Hacksore @PierreLevres
Unfortunately it turned out that the assumption is not right (no blame to your great work though!).

We have our Kia since 1.5 years now and since 1 year we have a lot of full outages of our 12 V battery.
Multiple Kia service partners did not find a root cause yet. But I did, since I finally bought a battery guard / monitor.
On each request to blulinky, of course with refresh=false, there is a short drop of voltage, which clearly shows that something happens in the car. I'm doing a request every hour. After 1 to 2 days the battery is completely empty. (Battery is probably damaged already and a fresh one could last longer).

So now the question is if this is somehow solvable. I think that my usage is correct. And I also assume its not a general problem, as you guys would be aware of it then.
Could you imagine that it is car model specific whether the blulinky request to the server is doing a refresh or not? I'm living in Europe and have a Niro EV SG2 from end of 2022.

The caching should happen at Kia side, not on blulinky side, correct?

Edit: Created a new issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants