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

Experimental: add heating support #6301

Closed
wants to merge 1 commit into from
Closed

Experimental: add heating support #6301

wants to merge 1 commit into from

Conversation

andig
Copy link
Member

@andig andig commented Feb 18, 2023

This PR provides initial support for dedicated heating units like "Heizstäbe".

TODO

Screenshot 2023-02-18 at 12 47 39

@andig andig requested a review from naltatis February 18, 2023 12:28
@andig andig marked this pull request as draft February 18, 2023 12:29
@andig andig added the enhancement New feature or request label Feb 18, 2023
@naltatis
Copy link
Member

Du hast hier den Fahrzeugwechsel (Gastfahrzeug, ...) bewusst ausgebaut. Das ist sicher auch sinnvoll und vermutlich auch bei vielen Smart-Steckdosen-Anwendungsfällen das gewünschte Verhalten. Ich fänd gut, wenn wir das konfigurationsseitig abbilden. Es gibt ja schon die 1:1 Loadpoint-Vehicle-Zuordnung, da könnten wir dem Loadpoint eine Art fixedVehicle Eigenschaft hinzufügen die die Fahrzeugänderung durch UI und API unterbindet.

@andig

This comment was marked as resolved.

@andig
Copy link
Member Author

andig commented Feb 18, 2023

Du hast hier den Fahrzeugwechsel (Gastfahrzeug, ...) bewusst ausgebaut. Das ist sicher auch sinnvoll und vermutlich auch bei vielen Smart-Steckdosen-Anwendungsfällen das gewünschte Verhalten. Ich fänd gut, wenn wir das konfigurationsseitig abbilden. Es gibt ja schon die 1:1 Loadpoint-Vehicle-Zuordnung, da könnten wir dem Loadpoint eine Art fixedVehicle Eigenschaft hinzufügen die die Fahrzeugänderung durch UI und API unterbindet.

Das könnten wir auch vorab in einen eigenen PR raus ziehen wenn Du magst. Für die Switchsockets ist das schon umgesetzt. Fehlt die Integration ins Vehicle.vue. Wenn Du das hier ergänzen willst würde ich dann splitten.

Was hältst Du davon, dafür auch ein Icon am Charger definierbar zu machen? Spannend wird nochmal, wie ein Vehicle ohne Namen (da ohne Vehicle) dann eigentlich aussehen soll... Auch "verbunden" als Status muss entfallen.

api/api.go Outdated Show resolved Hide resolved
@naltatis
Copy link
Member

better name for fixedconnection (noplug?)

noplug find ich nicht so gut. Gerade weil Smartsteckdosen ja immer einen haben, man in der Konfiguration aber sagten will, dass man den nicht umstecken wird.

Wir könnten auch in Richtung withoutVehicle/noVehicle gehen. In den Szenarien nutzen wir ja immer das "Gastfahrzeug".

@naltatis
Copy link
Member

proper icon handling for energy flow

Für die UI kann das Icon eigentlich transparent gehandhabt werden. Also selbst wenn wir es am Charger pflegbar machen würde ich vorschlagen das in der API weiterhin als vehicleIcon bereitzustellen. Dann haben wir in der UI keine weiteren Sonderfälle. Und zwei Icon gleichzeitig wollen wir ja ohnehin nicht.

@naltatis
Copy link
Member

Das könnten wir auch vorab in einen eigenen PR raus ziehen wenn Du magst.

Finde ich eine gute Idee wenn das vom Aufwand vertretbar ist. Gerne auch erstmal nur die Go Änderungen und ich überleg mir dann den UI Part.

@andig
Copy link
Member Author

andig commented Feb 19, 2023

proper icon handling for energy flow

Für die UI kann das Icon eigentlich transparent gehandhabt werden. Also selbst wenn wir es am Charger pflegbar machen würde ich vorschlagen das in der API weiterhin als vehicleIcon bereitzustellen. Dann haben wir in der UI keine weiteren Sonderfälle. Und zwei Icon gleichzeitig wollen wir ja ohnehin nicht.

Mache ich so 👍🏻. Die Features würde ich gerne als charger features lassen, da es ja tatsächlich andere Fähigkeiten als am vehicle sind.

@andig
Copy link
Member Author

andig commented Feb 19, 2023

Und zwei Icon gleichzeitig wollen wir ja ohnehin nicht.

Ist umgesetzt. Damit bietet sich an, das in 2 PRs aufzusplitten:

@andig
Copy link
Member Author

andig commented Feb 21, 2023

Rebased onto master

@dc-redeclipse
Copy link

Bildschirm­foto 2023-03-10 um 14 30 29
Es geht um die bessere Visualisierung von Akkus z.B. Ecoflows oder eigenbau Akkuspeichern die als Hausspeicher genutzt werden sollen. Dazu gehört auch das die Dialoge nicht mehr z.B. "Warten auf Fahrzeuge" anzeigen.

@github-actions github-actions bot added the stale Outdated and ready to close label Apr 1, 2023
@andig andig reopened this Apr 30, 2023
@github-actions github-actions bot removed the stale Outdated and ready to close label May 1, 2023
@github-actions github-actions bot added the stale Outdated and ready to close label Jun 10, 2023
@andig andig added backlog Things to do later and removed stale Outdated and ready to close enhancement New feature or request labels Jun 12, 2023
@andig andig closed this Aug 15, 2023
@andig andig mentioned this pull request Aug 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog Things to do later
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants