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
| for (var transferRule : store.getAllEntitiesForType(FareLegRule.class)) { | ||
| transferRule.getFareProductId().setAgencyId(reader.getDefaultAgencyId()); | ||
| } |
There was a problem hiding this comment.
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