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

change input vehicle file for EventBasedAnalysis #3036

Closed
wants to merge 3 commits into from

Conversation

rewertvsp
Copy link
Contributor

Changes the input vehicleFile for the analysis.

  • the reason is that the allVehicles file only contains the types of the used vehicles

Problem: if we read a carrier which uses not all available vehicles the types of this vehicles are not in the allVehicles file, and it creates this exception
Solution:

  • using the carriersVehicleTypes file instead

@rewertvsp rewertvsp requested a review from kt86 January 5, 2024 11:34
@kt86
Copy link
Contributor

kt86 commented Jan 5, 2024

I would like to stay for the analysis with the "allVehicles" container.

If the CarrierReader is the problem, Then we may need the option to read the carriers file even, if some vehicleTypes are missing. This should not be the default because for the normal simulation and tour planning, we require all vehicle types, but here for the analysis, we do not.

@kt86
Copy link
Contributor

kt86 commented Jan 5, 2024

Will close this PR here.
I have created an issue instead, that should be addressed when implementing another approach.

@kt86 kt86 closed this Jan 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants