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

[Do not merge] DT-5921 Fix transfer cache requests #416

Draft
wants to merge 3 commits into
base: v3
Choose a base branch
from

Conversation

VillePihlava
Copy link
Contributor

This PR removes the BICYCLE_RENT transfer cache requests that were not used. They are replaced with BICYCLE transfer cache requests that have additional fields. The added fields are non-required information in theory, but are required by the current OTP implementation.

This PR also adds CAR transfer cache requests in the finland and waltti configs. These will be properly utilized once the features from this PR are used: opentripplanner/OpenTripPlanner#5966.

@VillePihlava
Copy link
Contributor Author

The car transfer requests should generate walk transfer requests because these requests use OTPs QualifiedModeSet and currently it is configured this way. This will be changed by me in a future pr to OTP.

@VillePihlava VillePihlava changed the title Fix transfer cache requests DT-5921 Fix transfer cache requests Oct 29, 2024
@VillePihlava VillePihlava marked this pull request as draft November 1, 2024 11:18
@VillePihlava VillePihlava changed the title DT-5921 Fix transfer cache requests [Do not merge] DT-5921 Fix transfer cache requests Nov 1, 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.

1 participant