Skip to content

Commit ad9b93b

Browse files
Merge remote-tracking branch 'upstream/dev-2.x' into flex-graph-path-perf
2 parents ceb2bce + c4952f0 commit ad9b93b

57 files changed

Lines changed: 590 additions & 357 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

application/src/main/java/org/opentripplanner/apis/gtfs/datafetchers/FeedImpl.java

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -66,6 +66,12 @@ public DataFetcher<String> feedId() {
6666
return this::getSource;
6767
}
6868

69+
@Override
70+
public DataFetcher<String> version() {
71+
return environment ->
72+
getTransitService(environment).getFeedInfo(getSource(environment)).getVersion();
73+
}
74+
6975
@Override
7076
public DataFetcher<FeedPublisher> publisher() {
7177
return environment -> {

application/src/main/java/org/opentripplanner/apis/gtfs/generated/GraphQLDataFetchers.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -383,6 +383,8 @@ public interface GraphQLFeed {
383383
public DataFetcher<Iterable<Agency>> agencies();
384384
public DataFetcher<Iterable<TransitAlert>> alerts();
385385
public DataFetcher<String> feedId();
386+
387+
public DataFetcher<String> version();
386388
public DataFetcher<FeedPublisher> publisher();
387389
}
388390

application/src/main/java/org/opentripplanner/routing/algorithm/raptoradapter/transit/mappers/RaptorRequestMapper.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -180,7 +180,7 @@ public RaptorRequest<T> mapRaptorRequest() {
180180

181181
if (raptorDebugging.isEnabled()) {
182182
var debug = builder.debug();
183-
var debugLogger = new SystemErrDebugLogger(true, false);
183+
var debugLogger = new SystemErrDebugLogger(null, true);
184184

185185
debug
186186
.withStops(mapDebugStopList(raptorDebugging.stops()))

application/src/main/resources/org/opentripplanner/apis/gtfs/schema.graphqls

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -690,6 +690,8 @@ type Feed {
690690
feedId: String!
691691
"The publisher of the input transit data."
692692
publisher: FeedPublisher
693+
"The version of the dataset."
694+
version: String
693695
}
694696

695697
"Feed publisher information"

application/src/test-fixtures/java/org/opentripplanner/model/FeedInfoTestFactory.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,6 @@
33
public class FeedInfoTestFactory {
44

55
public static FeedInfo dummyForTest(String id) {
6-
return new FeedInfo(id, "publisher", "www.z.org", "en", null, null, null);
6+
return new FeedInfo(id, "publisher", "www.z.org", "en", null, null, "12345");
77
}
88
}

application/src/test/resources/org/opentripplanner/apis/gtfs/expectations/feedinfo.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,8 @@
1111
"publisher": {
1212
"name": "publisher",
1313
"url": "www.z.org"
14-
}
14+
},
15+
"version": "12345"
1516
}
1617
]
1718
}

application/src/test/resources/org/opentripplanner/apis/gtfs/queries/feedinfo.graphql

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,5 +8,6 @@
88
name
99
url
1010
}
11+
version
1112
}
1213
}

doc/user/Changelog.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,8 @@ based on merged pull requests. Search GitHub issues and pull requests for smalle
1414
- fix: Introduce stable order for fareZones (tariffZones) [#7434](https://github.com/opentripplanner/OpenTripPlanner/pull/7434)
1515
- Prevent long routes mistakenly getting grouped across service-date [#7452](https://github.com/opentripplanner/OpenTripPlanner/pull/7452)
1616
- Migrate away from deprecated FeedScopedId.parse() [#7457](https://github.com/opentripplanner/OpenTripPlanner/pull/7457)
17+
- Expose GTFS feed version in the GTFS GraphQL API [#7445](https://github.com/opentripplanner/OpenTripPlanner/pull/7445)
18+
- Fix MC Raptor Pareto comparator performance regression [#7440](https://github.com/opentripplanner/OpenTripPlanner/pull/7440)
1719
[](AUTOMATIC_CHANGELOG_PLACEHOLDER_DO_NOT_REMOVE)
1820

1921
## 2.9.0 (2026-03-18)

raptor/src/main/java/org/opentripplanner/raptor/api/view/ArrivalView.java

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -140,11 +140,6 @@ default RaptorTransfer transfer() {
140140

141141
/* Egress */
142142

143-
/** @return true if destination arrival, otherwise false. */
144-
default boolean arrivedAtDestination() {
145-
return false;
146-
}
147-
148143
default EgressPathView egressPath() {
149144
throw new UnsupportedOperationException();
150145
}

raptor/src/main/java/org/opentripplanner/raptor/rangeraptor/RangeRaptor.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,8 @@
4040
* (generating randomized schedules)
4141
* </ul>
4242
* <p>
43-
* This class originated as a rewrite of Conveyals RAPTOR code: https://github.com/conveyal/r5.
43+
* This class originated as a rewrite of
44+
* <a href="https://github.com/conveyal/r5">Conveyals RAPTOR code</a>.
4445
*
4546
* @param <T> The TripSchedule type defined by the user of the raptor API.
4647
*/
@@ -84,7 +85,7 @@ public RangeRaptor(
8485
this.calculator = requireNonNull(calculator);
8586
this.timers = requireNonNull(timers);
8687
this.accessPaths = requireNonNull(accessPaths);
87-
this.minNumberOfRounds = accessPaths.calculateMaxNumberOfRides();
88+
this.minNumberOfRounds = accessPaths.maxNumberOfRides();
8889
this.roundTracker = requireNonNull(roundTracker);
8990
this.lifeCycle = requireNonNull(lifeCyclePublisher);
9091
this.timeoutHook = requireNonNull(timeoutHook);

0 commit comments

Comments
 (0)