demus/openhab-addons
1
0
Fork 0
mirror of https://github.com/openhab/openhab-addons.git synced 2025-01-25 14:55:55 +01:00
Code Issues Packages Projects Releases Wiki Activity

[tesla] Remove the minimum 5A charge current limit (#15711)

Browse Source 
While the app can't go lower than 5A, the API allows setting values down
to 0A, and at least going to 2-3A can make sense if you want to put
excess solar power into the car without also drawing some from the grid.

Due to the overhead this causes it can be wasteful, so keep logging the
warning about going below 5A.

Signed-off-by: Ulrich Spörlein <uspoerlein@gmail.com>
This commit is contained in:
uqs 2023-10-11 23:10:37 +02:00 committed by GitHub
parent 7f32499a8a
commit 2797e6f278
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
bundles/org.openhab.binding.tesla/src/main/java/org/openhab/binding/tesla/internal/handler/TeslaVehicleHandler.java
View File

@ -283,10 +283,13 @@ public class TeslaVehicleHandler extends BaseThingHandler {
}
}
if (amps != null) {
if (amps < 5 || amps > 32) {
logger.warn("Charging amps can only be set in a range of 5-32A, but not to {}A.", amps);
if (amps > 32) {
logger.warn("Charging amps cannot be set higher than 32A, {}A was requested", amps);
return;
}
if (amps < 5) {
logger.info("Charging amps should be set higher than 5A to avoid excessive losses.");
}
setChargingAmps(amps);
}
break;