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: Add a CarrierJob interface #3599

Merged
merged 3 commits into from
Dec 2, 2024
Merged

Conversation

kt86
Copy link
Contributor

@kt86 kt86 commented Nov 27, 2024

and put CarrierService and CarreirShipment behind it.

This is a first steps on the way to

  1. store the Jobs, nomatter wheter it is a Service or a Cshipment (helpfull for logistics)
  2. It is a first step, to reduce code, by putting more and more commen things in here and
  3. allign CarrierServices and CarrierShipments a bit more
    See also issue: Freight: Introduce something like a "CarrierJob" on MATSim-level #3510

@JWJoubert, @rewertvsp: I added you as reviewers, so you can a) get an info about this change and b) can comment on the name of the interface. 'CarrierJob' was just my first idea.

@kt86 kt86 requested review from JWJoubert and rewertvsp November 27, 2024 12:19
@kt86 kt86 self-assigned this Nov 29, 2024
@kt86 kt86 enabled auto-merge December 2, 2024 14:50
@kt86 kt86 merged commit 166a09f into master Dec 2, 2024
49 checks passed
@kt86 kt86 deleted the kmt_addCarrierJobInterface branch December 2, 2024 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

Freight: Introduce something like a "CarrierJob" on MATSim-level
2 participants