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

Element „AutomaticPreconditioningType“ unvollständig #5

Closed
TNeuHochbahn opened this issue Mar 31, 2022 · 7 comments · Fixed by #6
Closed

Element „AutomaticPreconditioningType“ unvollständig #5

TNeuHochbahn opened this issue Mar 31, 2022 · 7 comments · Fixed by #6
Assignees
Labels
accepted bug Issue in interface or specification ready Ready to Merge specification Improvements or additions to specification
Milestone

Comments

@TNeuHochbahn
Copy link

Aus unserer Sicht fehlt im Element „AutomaticPreconditioningType“ analog zum beschriebenen Element "ManualPreconditioningType" das Feld welches den Startzeitpunkt der Vorkonditionierung "SystemPreconditioningStartTime" übermittelt. Ohne einen Startzeitpunkt kann das LMS nicht die benötigte Leistung rechtzeitig vor dem Abfahrtszeitpunkt/ dem Beginn der Vorkonditionierung bereitstellen.

Vorschlag: Entsprechende Felder im "AuotmaticPreconditionType" ergänzen.

@TNeuHochbahn TNeuHochbahn changed the title Element „AutomaticPreconditioning“ unvollständig Element „AutomaticPreconditioningType“ unvollständig Mar 31, 2022
@maikwoehl-psi maikwoehl-psi added the proposed A proposal to interface change label Apr 1, 2022
@hojatzarif
Copy link

Die automatische Vorkonditionierung wird über die VDV261 Kommunikation zwischen Bus und LMS umgesetzt. Dabei gibt der Bus den Startzeitpunkt der Vorkonditionierung vor. Vom Vorsystem über das LMS wird nur der Wunsch nach einer Vorkonditionierung, die Art der Vorkonditionierung und die Endzeitpunkt der Vorkonditionierung übermittelt. Den Beginn der Vorkonditionierung bestimmt der Bus selbstständig.

@mafrenzelpsi
Copy link
Collaborator

Eine SystemPreconditioningStartTime, wie von @TNeuHochbahn vorgeschlagen, wird m.E. dann erforderlich, wenn die Ladeleistung nach Erreichung eines Zielladezustandes auf eine minimale Erhaltungsladung reduziert wurde und für die PreKonditionierung wieder erhöht werden muß. Alternativ könne man auf den Gedanken kommen die von Fahrzeug berechnete und per VDV 261 übertragene Zeit zur Vorkonditionierung zur verwenden (prec_reqtime), die Frage ist nur ob diese hilfreich ist, wenn die Berechnung auf Basis der reduzierten minimale Erhaltungsladung erfolgt.

@maikwoehl-psi
Copy link
Collaborator

So würde ich also folgende Ergänzung vorschlagen:

"AutomaticPreconditioning": {
	"description": "This type imposes properties regarding automatic preconditioning.",
	"type": "object",
	"properties": {
		"preconditioningRequest": {
			"$ref": "#/definitions/PreconditioningRequest"
		},
		"ambientTemperature": {
			"type": "number",
			"additionalProperties": false
		},
		"requestedStartTime": {
			"type": "string",
			"format": "date-time",
			"additionalProperties": false
		}
		"requestedFinishTime": {
			"type": "string",
			"format": "date-time",
			"additionalProperties": false
		}
	},
	"required": [
		"preconditioningRequest"
	],
	"additionalProperties": false
},

@mafrenzelpsi mafrenzelpsi self-assigned this Jun 28, 2022
@mafrenzelpsi
Copy link
Collaborator

mafrenzelpsi commented Jun 28, 2022

Die vorgeschlagene Änderung wurde in der VDV 463 Arbeitsteamsitzung am 28.6.2022 akzeptiert. Das Attribut "requestedStartTime" soll als optionales Attribut im AutomaticPreconditioningType aufgenommen werden. Die Dokumentation soll ergänzt werden, so dass der Anwendungsfall daraus ersichtlich wird.

@maikwoehl-psi maikwoehl-psi added specification Improvements or additions to specification and removed proposed A proposal to interface change labels Jul 4, 2022
maikwoehl-psi added a commit to maikwoehl-psi/VDV463 that referenced this issue Jul 4, 2022
@maikwoehl-psi maikwoehl-psi added this to the Version 1.1 milestone Jul 4, 2023
@maikwoehl-psi
Copy link
Collaborator

maikwoehl-psi commented Oct 25, 2023

  • Änderung des Spezifikationsdokument noch notwendig?

@maikwoehl-psi maikwoehl-psi added the ready Ready to Merge label Jan 18, 2024
@maikwoehl-psi maikwoehl-psi self-assigned this Jan 18, 2024
@DohmenClaus DohmenClaus added the bug Issue in interface or specification label Feb 26, 2024
@DohmenClaus
Copy link
Contributor

Abbildung in Kap. 2.2.3 muss noch angepasst werden

@maikwoehl-psi
Copy link
Collaborator

maikwoehl-psi commented Jul 1, 2024

Im Arbeitsgruppen-Termin am 01.07.2024 hat sich die bereits im Spezifikationsdokument eingearbeitete Erläuterung, dass das Vorsystem in der Lage sein soll für die automatische Vorkonditionierung den Zeitpunkt zu übermitteln, bei dem die Leistung zur Vorkonditionierung vom LMS zur Verfügung gestellt werden soll, weiterhin Bestand hat und um eine Erläuterung zu Fahrzeugen ohne VDV261-Unterstützung ergänzt werden soll, die darauf hinweist, dass die Startzeit dann auch im Vorkonditionierungsprotokoll mit dem Fahrzeug als tatsächliche Startzeit der Vorkonditionierung verwendet wird.

  • Die Erläuterung zur requestedStartTime in dem Spezifikationsdokument soll besser ausgeführt werden.
  • Hinweis auf die Nutzung als tatsächliche Startzeit der Vorkonditionierung bei Fahrzeugen ohne VDV261-Unterstützung zum Spezifikationsdokument hinzufügen.
  • Neues Issue: Ergänzung des Architektur-Teils des Spezifikationsdokument, um eine grobe Beschreibung der Verbindung zwischen Schnittstelle und Stammdaten, die von außen kommen. (@DohmenClaus) Erläuterung zu Stammdaten der beteiligten Systeme #33

maikwoehl-psi added a commit that referenced this issue Dec 2, 2024
* Add v1.1 JSON schemas fixing issues #2 and #3

* Fixes #5

* removing "uri" format constraint from all string identifiers

as agreed in the VDV 463 meeting

* removing "uri" format constraint from all string identifiers

* removing "uri" format constraint from all string identifiers

* Remove obsolete whitespace changes

* #20: Allow empty arrays

* #22: Allow empty ChargingPredictionData

* #22: Add structure ChargingPredictionDataDepartureTime to ChargingPredictionData

* #22: Fix descriptions

* Update v1.1/ProvideChargingInformationRequest.json

Co-authored-by: Maik Wöhl <[email protected]>

* Update v1.1/ProvideChargingInformationRequest.json

Co-authored-by: Maik Wöhl <[email protected]>

* Add critical condition for tractionBattery, add hydrogentankinfo (#16)

* Add critical condition for tractionBattery, add hydrogentankinfo

* Move vehicleInfo into it's own list

* Add VehicleId to ChargingPointInfo to reference it

* Switch criticalCondition to integer to reflect SNA values

* Revert "Add VehicleId to ChargingPointInfo to reference it"

This reverts commit d14814e.

Revert "Move vehicleInfo into it's own list"

This reverts commit 0aa89ce.

* Rename criticalCondition to status

* Revert "Add v1.1 JSON schemas fixing issues #2 and #3"

* Add evccId to VehicleInfo structure

Closes #30

* Revert "Add critical condition for tractionBattery, add hydrogentankinfo (#16)"

This reverts commit c82bf61.

* Add Validations to JSON Schema Properties (#36)

* Move changes of v1.1 to v1.0

Signed-off-by: Maik Wöhl <[email protected]>

* Rename v1.0 to schema

Signed-off-by: Maik Wöhl <[email protected]>

---------

Signed-off-by: Maik Wöhl <[email protected]>
Co-authored-by: Kowatsch, Benjamin <[email protected]>
Co-authored-by: Dominique Müller <[email protected]>
Co-authored-by: Benjamin Kowatsch <[email protected]>
Co-authored-by: DohmenClaus <[email protected]>
Co-authored-by: Kilian Schneider <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accepted bug Issue in interface or specification ready Ready to Merge specification Improvements or additions to specification
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants