Upstream merge 2022-09-23 #128
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…o timetableUpdates in config These parameters are not updater parameters and configure the realtime updates in general. Currently, OTP can run with either a Siri or a GTFS SnapshotProvider. These parameters are passed to the provider. This enables us to inject the correct provider into the injector instead of duplicating the snapshot provider creation logic in every updater.
Co-authored-by: Hannes Junnila <[email protected]>
Added in in one place, where it was not previously checked
Collections.emptyList returns the same instance with a static iterator
Performance improvements for DefaultFareServiceImpl
…ean_code doc: Document why unused parameters in Raptor API should NOT be deleted.
…eAndInterpolateStopTimesForEachTrip.java Co-authored-by: Thomas Gran <[email protected]>
…stoptimes Improve limits in ValidateAndInterpolateStopTimesForEachTrip
…apper Do not crash if PageCursorFactory can't be created
In this refactoring all initialization is moved into the Updater constructor. Only used dependencies are injected, as specific as needed. Example, the VehicleParkingUpdater does not need Graph and TransitModel, it needs a VertexLinker and VehicleParkingService, so those are injected. This make it possible to delete some updater fields(only needed for initialization) and to make some fields final.
…nitialization Catch exceptions from updater configuration
…rences Otp2 cleanup preferences - Part 1
…h_on_stops Add feature switch for matching ET messages on stops
Update code conventions doc on builders
…apper Take into account null values for fare details
…d_networks Allow specifying allowed/banned rental networks in the Transmodel API
…tenation Use placeholders in log messages
More graph stats, better exception message
|
Do you know if the way to load a vehicle positions feed changed? I'm getting some errors on router configs that previously worked:
|
|
I will investigate this. In the meanwhile I will add it to the smoke test. |
|
Thanks so much! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This brings IBI up to date with the latest upstream. It's only been roughly 6 weeks but there are over 800 commits! I should have done that earlier!
Changes relevant to IBI
To configure what used to be called a way property set, use the following in build-config.json:
And couple of bug fixes:
Improvement to the developer experience