-
-
Notifications
You must be signed in to change notification settings - Fork 915
template: add vehicle-features to vw #21606
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @StefanSchoof - I've reviewed your changes and they look great!
Here's what I looked at during the review
- 🟢 General issues: all looks good
- 🟢 Security: all looks good
- 🟢 Testing: all looks good
- 🟢 Complexity: all looks good
- 🟢 Documentation: all looks good
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
Aktuell gibts das nur wo benötigt. Schade, dass VW da einen Rückschritt macht :( |
Hat mein ID.3 aus 2020 nie gekonnt :( |
VW will Tesla-Level erreichen 😄 |
Wie nutze ich das Feature in yaml? Unter vehicles VW finde ich keine Beschreibung dazu |
Ist noch nur im nigthly. So sieht es bei mir jetzt aus: - type: template
template: vw
title: ID.3
user: <mail>
password: <pw>
vin: <vin>
capacity: 58
mode: pv
name: ID3
mincurrent: 6
coarsecurrent: |
Ich nutze seit Jahren zwei VW ID-Modelle (ID.3 & ID.5) mit evcc – ohne explizite Angabe von minCurrentStep. Die Fahrzeuge verhalten sich dabei absolut stabil und folgen zuverlässig den 1A-Stufen. Ich verstehe den Zweck der Option, halte die Diskussion aber für überbewertet – vor allem der ständige Vergleich mit Tesla nervt. Für PV-Überschussladen reicht 1A-Granularität völlig aus. Ich bin froh, dass VW hier einfach funktioniert – ohne unerwartetes Verhalten und ohne unnötige Komplexität. |
Das finde ich ein sehr interessanten Aspekt 😊 erfolgt die Steuerung der 1A Stufen über die Wallbox oder über das Auto? |
Nein geht um die ganz normale Steuerung der Wallbox. Die Wallbox gibt dem Auto vor wie viel Ampere es ziehen darf und es gibt Autos die das nur auf Ampere genau machen und andere auf Milliampere. |
@mj6517 aber auch nicht mehr. Was es anscheinend nicht kann (und v.a. bei 3p und kleiner PV sehr hilfreich ist) ist eine mA-Vorgabe zu verarbeiten. |
Meine Wahrnehmung ist: mein VW kann auch Schritte unterhalb von 1A, allerdings kann die Elli Wallbox es nicht - mit der Keba Wallbox werden 0,1A Schritte geregelt, mit der Elli Wallbox nur 1A Schritte… scheint mir daher - zumindest in meiner Konstellation - eher eine Limitierung der Wallbox (bei der man es leider nicht konfigurieren kann) als des Fahrzeugs zu sein… |
Mein ID.3 (SW 3.7) kann keine Milliampere Schritte. In meiner yaml hatte ich bisher daher
type: vw
. Das geht in der Config UI nicht.Mir ist dabei aufgefallen, dass einige Fahrzeug APIs
vehicle-features
haben (z.B. bmw und dacia), aber viele nicht.Soll es
coarsecurrent
undwelcomecharge
nur für Hersteller wo es bekannt ist, dass Autos dies benötigen? Soll es dann immer beides geben oder nur das wo bekannt ist, dass der Hersteller Fahrzeuge hat die dies benötigen?