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

Support chargers without vehicle #6316

Merged
merged 8 commits into from
Feb 21, 2023
Merged

Support chargers without vehicle #6316

merged 8 commits into from
Feb 21, 2023

Conversation

andig
Copy link
Member

@andig andig commented Feb 19, 2023

Chargers without vehicle are implemented using

features: ["integrateddevice"]

This will suppress vehicle detection and instruct the UI to hide the vehicle selection. Useful e.g. for heating units or socket chargers that switch a single device only.

@andig andig requested a review from naltatis February 19, 2023 15:08
@andig andig added the enhancement New feature or request label Feb 19, 2023
@andig andig mentioned this pull request Feb 19, 2023
5 tasks
@andig andig force-pushed the feature/fixed-connection branch from d1c11d8 to 9dcdb93 Compare February 19, 2023 16:29
@andig andig changed the title Allow chargers without vehicle Support chargers without vehicle Feb 19, 2023
@naltatis
Copy link
Member

Wie wollen wir das im Ladelog abbilden? Dort gibts ja die Spalte Fahrzeug, die könnten wir einfach leer lassen. Es gibt allerdings auch die Bearbeitenansicht. Dort würde ich die Fahrzeugauswahl auch gerne entfernen. Dafür bräuchten wir an der Stelle aber auch die LP-Information oder einen anderen Indikator.

@andig
Copy link
Member Author

andig commented Feb 19, 2023

Mhhm, das ist fies. Ich würds leer lassen und ignorieren. Dass jemand das Ladelog „fälscht“ unterdrücken wir ja auch heute nicht. Es taucht ja unter separatem Ladepunkt auf. Den Charger speichern wir allerdings nicht.

@naltatis
Copy link
Member

Mir gefällt das alleinstehende Fahrzeug-Icon ohne Beschriftung im Loadpoint nicht. Ich hab unterschiedliche Sachen ausprobiert und bislang nichts schönes gefunden. Daher tendiere ich jetzt gerade dazu, dass wir doch das Icon am Loadpoint anzeigen. Das hat den Vorteil, dass die Konfiguration dann auch näher an der Anzeige ist. Der Nachteil ist, dass wir ggf. einen Weißbereich an der Stelle haben wo sonst Icon/Fahrzeugname stehen. Ich würde das so bauen, dass dieser Freiraum entfällt wenn man nur Loadpoints ohne Fahrzeug hat. Sobald aber einer der Nachbar-LPs ein Fahrzeug hat wäre mir hier die Bündigkeit (Kästen gleich hoch, Ladebalken auf gleicher Höhe) wichtiger und würde den Freiraum akzeptieren.

Alleine bzw. nur LPs ohne Fahrzeug
Bildschirm­foto 2023-02-19 um 22 40 38

Mit Nachbar-LPs
Bildschirm­foto 2023-02-19 um 22 39 46

Heißt wir sollten das API-Datenmodell doch noch mal umstellen und das Charger-Icon auch als eigenes Feld an die UI geben. Die Visualisierung würde ich so umstellen, dass wir anstatt des Vehicle-Icons ggf. auch das Charger-Icon anzeigen.
Was meinst du?

@andig
Copy link
Member Author

andig commented Feb 20, 2023

Ist schnell geändert. Das schafft dann beim Heater vllt auch neue Darstellungsmöglichkeiten für den soc/temp.

@naltatis
Copy link
Member

Hier jetzt das angepasste Layout.

Bildschirm­foto 2023-02-20 um 22 46 37

Bildschirm­foto 2023-02-20 um 22 46 08

@andig
Copy link
Member Author

andig commented Feb 21, 2023

Einen hätte ich noch: IntegratedVehicle -> IntegratedDevice

@andig andig merged commit d351709 into master Feb 21, 2023
@andig andig deleted the feature/fixed-connection branch February 21, 2023 14:14
@StevieC121176
Copy link

StevieC121176 commented Feb 22, 2023

Cooles Feature. Eine Frage, was muss ich in der yaml ändern damit ich meinen Loadpoint ohne Fahrzeugauswahl angezeigt bekomme?

@andig
Copy link
Member Author

andig commented Feb 22, 2023

Geht nur am Charger. Welchen hast du denn?

@StevieC121176
Copy link

StevieC121176 commented Feb 22, 2023

Diesen hier:

chargers:
- type: template
  template: homematic 
  host: 192.168.178.68  
  port: 2010  
  device: '00****'  
  standbypower: 15  
  user: ****  
  password: '*****'  
  meterchannel: 6  
  switchchannel: 3  
  name: wallbox10

@andig
Copy link
Member Author

andig commented Feb 23, 2023

Das ist dann #6368

@evcc-io evcc-io locked as resolved and limited conversation to collaborators Feb 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants