Skip to content

Add support for creating constrained transfers from real-time generated patterns - #3878

Merged
hannesj merged 3 commits into
opentripplanner:dev-2.xfrom
entur:otp2_realtime_constrained_transfers
Feb 11, 2022
Merged

hannesj merged 3 commits into
opentripplanner:dev-2.xfrom
entur:otp2_realtime_constrained_transfers

Conversation

@hannesj

@hannesj hannesj commented Feb 9, 2022

Copy link
Copy Markdown
Contributor

Summary

Add support for creating constrained transfers from real-time generated patterns.

Issue

closes #3877

Unit tests

Not updated

Code style

Documentation

No changes required.

Changelog

The changelog file
is generated from the pull-request title, make sure the title describe the feature or issue fixed.
To exclude the PR from the changelog add [changelog skip] in the title.

@hannesj hannesj added the !Bug Apply to issues describing a bug and PRs witch fixes it. label Feb 9, 2022
@hannesj hannesj added this to the 2.1 milestone Feb 9, 2022
@hannesj
hannesj requested a review from t2gran February 9, 2022 07:13
@hannesj
hannesj requested a review from a team as a code owner February 9, 2022 07:13
@hannesj hannesj changed the title Otp2 realtime constrained transfers Add support for creating constrained transfers from real-time generated patterns Feb 9, 2022

@t2gran t2gran left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This now looks much better, a few comments on the RT update strategy and why this is now thread safe, is all I think is needed.

…nsit/constrainedtransfer/TransferIndexGenerator.java
@hannesj
hannesj merged commit 289e691 into opentripplanner:dev-2.x Feb 11, 2022
@hannesj
hannesj deleted the otp2_realtime_constrained_transfers branch February 11, 2022 13:55
t2gran pushed a commit that referenced this pull request Feb 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

!Bug Apply to issues describing a bug and PRs witch fixes it.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Constrained transfers should be copied to real-time patterns

3 participants