You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Die (LSP) Shipments haben derzeit neben ihrem Schedule auch einen eigenen Log. Dieser soll darstellen, was tatsächlich passiert.
Mit zunehmender Nutzung der MATSim-Event Infrastruktur ist dann die Frage, ob nicht diese die Grundlage für alles Weitere seien sollte und nicht so ein "zwischendurch" irgendwie erstelltes Log.
Das würde auch die Anzahl der Objekte deutlich reduzieren, die dafür dann nochmal als Teil-Dubletten angelegt werden...
Das impliziert dann auch, dass die entsprechenden Tests umgebaut werden auf Events-Vergleich -> MATSimTestUtils.compareEventsFile(...)
(Update 24.11.2022 KMT)
Alle dies betreffenden Tests erweitern um Events-Vergleich
dabei eventuell ein paar inhaltliche Checks in Assertion beibehalten?
ShipmentLog entfernen
Schauen, was dann in den EventHandler alles an Infrastruktur entfernt werden kann.
Had another discussion with KN: We will tread that log somehow as an experienced plan of the shipment.
So it is ok, that it remains.
Will check if we can improve some names, to clarify this.
Moreover, there will be some cleanup / follow-up work to e.g, to separate this “logging” from the productive code a bit. One first idea is to move as much as possible an own class, which will do all the logging in one place instead of having it spread all over the places.
((The logistics repo goes into archive mode, code is not on matims-libs. So the issue is moved from - see there for more information: matsim-vsp/logistics#50 ))
The text was updated successfully, but these errors were encountered:
Die (LSP)
Shipment
s haben derzeit neben ihremSchedule
auch einen eigenenLog
. Dieser soll darstellen, was tatsächlich passiert.Mit zunehmender Nutzung der MATSim-Event Infrastruktur ist dann die Frage, ob nicht diese die Grundlage für alles Weitere seien sollte und nicht so ein "zwischendurch" irgendwie erstelltes Log.
Das würde auch die Anzahl der Objekte deutlich reduzieren, die dafür dann nochmal als Teil-Dubletten angelegt werden...
Das impliziert dann auch, dass die entsprechenden Tests umgebaut werden auf Events-Vergleich ->
MATSimTestUtils.compareEventsFile(...)
(Update 24.11.2022 KMT)
Had another discussion with KN: We will tread that log somehow as an experienced plan of the shipment.
So it is ok, that it remains.
Will check if we can improve some names, to clarify this.
Moreover, there will be some cleanup / follow-up work to e.g, to separate this “logging” from the productive code a bit. One first idea is to move as much as possible an own class, which will do all the logging in one place instead of having it spread all over the places.
((The logistics repo goes into archive mode, code is not on matims-libs. So the issue is moved from - see there for more information: matsim-vsp/logistics#50 ))
The text was updated successfully, but these errors were encountered: