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

Freight: Combine LSP and Carrier Shipment? #3610

Open
1 of 4 tasks
kt86 opened this issue Nov 29, 2024 · 0 comments
Open
1 of 4 tasks

Freight: Combine LSP and Carrier Shipment? #3610

kt86 opened this issue Nov 29, 2024 · 0 comments

Comments

@kt86
Copy link
Contributor

kt86 commented Nov 29, 2024

It is very confusing for me, because we have a lot of usage of LSPShipments, but in freight there are existing the CarrierShipments

In many steps, e.g. in the ScheduledShipmentLoad, ScheduledShipmentHandle, ScheduledShipmentUnload, a Carrier_Service_ is added .... (which is neither a LSPShipment not a CarrierShipment.

If we decide in the future to also use CarrierShipments (see matsim-vsp/logistics#44), the confusion is perfect :(

Maybe try in a first step to name everything that is related to an LSPShipment with lspShipment (instead of just shipment) ?? (Or do we already have a better idea? --> "job", "order", ... ?

(Update 24.11.2022)

  • Rename all shipments entsprechend ihres Types -> lspShipment, carrierShipment
  • Prüfe Datenstuktur von LSPShipment und CarrierShipment und wie man die zur Deckung bringen könnte, ggf. mit Attributen.
  • Diskussion und Entscheidung mit KN: Führen wir das zusammen oder sagen wir, dass der LSP "Aufträge" / "Jobs" bekommt und die Shipments dann das sind, was zwischen den Carriern rumgereicht wird - da sind ja u.U interne Zeitfenster anders.
  • Umsetzung dessen

After discussion witth KN:

Let's check, if we there is enough overlap between LSPShipment and CarrierShipment so we can condense it to just one type of Shipments.

If some fields are missing, we can use Attributes for it.

((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#45 ))

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant