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

Add CoarseCurrent feature for vehicles that can't follow mA charging #4662

Merged
merged 5 commits into from
Oct 5, 2022

Conversation

andig
Copy link
Member

@andig andig commented Sep 30, 2022

Depends on #4639

Problem: Fahrzeug (Tesla) kann nur in ganzen Ampere regeln; evcc "denkt" aber an entsprechenden Chargern, das Auto würde auch kleinen Änderungen folgen (was es nicht tut).
Ergebnis: bei wenigen mA Änderung von x,4 auf x,6 Ampere springt die Leistung um 1A mal Phasen. Die Steuerung fängt an zu takten.

Lösung:

add

features: ["coarsecurrent"]

to the vehicle config.

/cc @VolkerK62

@andig andig added the enhancement New feature or request label Sep 30, 2022
@andig andig requested a review from premultiply September 30, 2022 22:07
@premultiply
Copy link
Member

Huch, ist da noch mehr in diesen PR gerutscht?

@andig
Copy link
Member Author

andig commented Sep 30, 2022

Nee, das liegt nur dran, dass der andere noch nicht gemerged ist und ich die Features dafür brauche.

@VolkerK62
Copy link
Contributor

Lösung:
add
features: ["coarsecurrent"]
to the vehicle config.
/cc @VolkerK62

und was trägt man da ein? true/false?

@andig andig marked this pull request as draft October 1, 2022 08:08
@andig andig force-pushed the feature/coarse-current branch from 39cb58c to 5c9bd9d Compare October 5, 2022 08:45
@andig andig marked this pull request as ready for review October 5, 2022 08:45
@andig andig merged commit 6e943a0 into master Oct 5, 2022
@andig andig deleted the feature/coarse-current branch October 5, 2022 15:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants