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

chore: default zoning params in DvrpTravelTimeMatrixParams #3483

Merged

Conversation

tkchouaki
Copy link
Contributor

@nkuehnel

In the DvrpConfigGroup class, the getTravelTimeMatrixParams() method creates a DvrpTravelTimeMatrixParams on the fly if it doesn't already exist (meaning it wasn't in the XML input or it wasn't added by the script that created the DvrpConfigGroup).

However, when a DvrpTravelTimeMatrixParams is created in the above mentioned method, no ZoneSystemParams is created inside it. Which then causes a NullPointerException whenever the ZoneSystemParams is requested and used.

To address this, this PR continues the same approach of creating the params on the fly by creating a SquareGridZoneSystemParams with a cellSize of 200m, as was the default in the old implementation.

@nkuehnel
Copy link
Member

LGTM, thanks!

@jfbischoff jfbischoff merged commit 22d8b7e into matsim-org:master Sep 16, 2024
47 checks passed
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.

3 participants