-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Update to newest version of GTFS Flex location groups #5655
Update to newest version of GTFS Flex location groups #5655
Conversation
950a8fb
to
4b5a43b
Compare
4b5a43b
to
a16d3c1
Compare
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## dev-2.x #5655 +/- ##
=============================================
+ Coverage 67.62% 67.64% +0.02%
- Complexity 16290 16309 +19
=============================================
Files 1886 1887 +1
Lines 71542 71579 +37
Branches 7384 7390 +6
=============================================
+ Hits 48378 48418 +40
+ Misses 20660 20656 -4
- Partials 2504 2505 +1 ☔ View full report in Codecov by Sentry. |
I've converted this PR to a draft as Arcadis has requested that some backwards-compatibility remains. My goal is to support two types of location groups:
Since this is quite a confusing topic, can you confirm that this plan is what we agreed, @miles-grant-ibigroup, @br648? |
@leonardehrenfried This sound about right to me. The very latest (new location groups) and the last iteration (Area and StopArea). For clarity, data tools is still on the last iteration and will be migrated to the latest once the spec draft has been approved. |
c60a490
to
1a83868
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
only minor remarks
src/main/java/org/opentripplanner/gtfs/mapping/StopAreaMapper.java
Outdated
Show resolved
Hide resolved
src/main/java/org/opentripplanner/gtfs/mapping/StopAreaMapper.java
Outdated
Show resolved
Hide resolved
src/main/java/org/opentripplanner/gtfs/mapping/StopTimeMapper.java
Outdated
Show resolved
Hide resolved
for (var transferRule : store.getAllEntitiesForType(FareLegRule.class)) { | ||
transferRule.getFareProductId().setAgencyId(reader.getDefaultAgencyId()); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is this change?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Setting the correct agency id, which we convert to a feed id, was wrong before.
Summary
In the recent Flex working group meeting we decided to slightly change the semantics of stop locations: google/transit#388 (comment)
The original change in OBA is here: OneBusAway/onebusaway-gtfs-modules#227
Backwards compatibility
Since Arcadis tooling is still outputting the previous iteration of the spec (stop areas), I added some backwards compatibility code for that. Once the spec has been approved I will remove it and other backwards-compatibility code.
GTFS files
You will notice that this PR adds lots of new files to the repo. This actually only unzips the GTFS zips that were there before so no new (slow!) integration test has been added.
Unit tests
Added.
cc @gcamp @westontrillium @tzujenchanmbd @eliasmbd