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

Lower EV Smart Charging Minimum EV SOC does not change duration of expected charging #264

Open
Ferrox85 opened this issue May 17, 2024 · 10 comments
Labels
bug Something isn't working

Comments

@Ferrox85
Copy link

Version 1.10.0

If for example 50 % of charging is set as goal. I expect that the duration is lowered with 50%. For example from 10 (normally 100%) to 5 hours and that it recalculates the most efficient hours to charge with those 5 hours.

@jonasbkarlsson
Copy link
Owner

Well, it depends on the current SOC. If your current SOC is 0%, then your example is what will happens. But if you for example have 25% SOC and it would take 10 hours to reach 100% (from 25%), then if you change the target SOC to 50%, then it will take (50-25)/(100-25)*10 = 3.33 hours (from 25% to 50%), which the integration rounds up to 4 hours.

@Ferrox85
Copy link
Author

Exactly, but it doesn't do recalculation that when I change the target. So it looks like a bug right?

@yoali11
Copy link

yoali11 commented Jun 25, 2024

It is the same for me. Whenever the target soc slider is changed nothing is recalculated. I have to manually turn off and back on the smart charge enable switch to have the integration recalculate a new charge plan.
Actually I have an automation that does this for me.
Would be nice to have this embedded natively in the integration.

@jonasbkarlsson
Copy link
Owner

Ah, ok. I will check.

@jonasbkarlsson jonasbkarlsson added the bug Something isn't working label Jun 26, 2024
@jonasbkarlsson
Copy link
Owner

In v1.11.0, changing the target SOC will update the charging plan.

Was that the issue, or was there also something else? Hmm. Is it the same problem when changing Minimum EV SOC?

@Ferrox85
Copy link
Author

I don't use the minimum setting, but maybe? I can't test it right now..

@jonasbkarlsson
Copy link
Owner

Now I have tested. Changing Minimum EV SOC will update the charging plan. So then I consider this issue closed.

@Ferrox85
Copy link
Author

Just updated to this version, it does not fix the issue. I change the entity "EV Smart Charging Minimum EV SOC" Sorry for the inconvenience, I thought it was an other entity. This is because of some coupled automations and helpers in my HASS.

If I change continues charging for example, it immediately recalculates. But changing the EV Smart Charging Minimum EV SOC entity does not update.

@jonasbkarlsson
Copy link
Owner

@Ferrox85, I'm not able to recreate the issue. Changes in number.ev_smart_charging_minimum_ev_soc and switch.ev_smart_charging_continuous_charging_preferred should both trigger recalculation, or neither of them. Note that recalculation is not done if current time is after select.ev_smart_charging_charge_completion_time (if not set to None) and the tomorrow's prices are not available.

What other settings do you have for the integration, and at what time of the day do you see this issue?

@Ferrox85
Copy link
Author

If I am able to try again I will report back. Currently our car is full...

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