Skip to content

Commit

Permalink
speed up transit ingestion (valhalla#4167)
Browse files Browse the repository at this point in the history
* remove just_gtfs official version

* use sorted feeds to speed up processing

* changelog

* use new no copy interfaces with different validity semantics

* lint changelog

* lint

* fix one of the test failures but the calendar part doesnt make sense..

* fix last bug, ready to ship

* fix yet another range bug. i see why c++20 made these first class citizens of stl
  • Loading branch information
kevinkreiser authored Jul 4, 2023
1 parent b81a6c2 commit 30e11f1
Show file tree
Hide file tree
Showing 6 changed files with 96 additions and 94 deletions.
2 changes: 1 addition & 1 deletion .gitmodules
Original file line number Diff line number Diff line change
Expand Up @@ -45,4 +45,4 @@
url = https://github.com/jarro2783/cxxopts.git
[submodule "third_party/just_gtfs"]
path = third_party/just_gtfs
url = https://github.com/mesozoic-drones/just_gtfs.git
url = https://github.com/valhalla/just_gtfs
3 changes: 2 additions & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@
* ADDED: CI runs a spell check on the PR to detect spelling mistakes [#4179](https://github.com/valhalla/valhalla/pull/4179)
* ADDED: `preferred_side_cutoff` parameter for locations [#4182](https://github.com/valhalla/valhalla/pull/4182)
* ADDED: PBF output for matrix endpoint [#4121](https://github.com/valhalla/valhalla/pull/4121)
* CHANGED: sped up the transit gtfs ingestion process by sorting the feeds before querying them and avoiding copying their structures. forked just_gtfs into the valhalla org to accomplish it [#4167](https://github.com/valhalla/valhalla/pull/4167)

## Release Date: 2023-05-11 Valhalla 3.4.0
* **Removed**
Expand Down Expand Up @@ -97,7 +98,7 @@
* ADDED: Added has_toll, has_highway, has_ferry tags to summary field of a leg and route and a highway tag to a maneuver if it includes a highway. [#3815](https://github.com/valhalla/valhalla/issues/3815)
* ADDED: Add time info to sources_to_targets [#3795](https://github.com/valhalla/valhalla/pull/3795)
* ADDED: "available_actions" to the /status response [#3836](https://github.com/valhalla/valhalla/pull/3836)
* ADDED: "waiting" field on input/output intermediate break(_through) locations to respect services times [#3849](https://github.com/valhalla/valhalla/pull/3849)
* ADDED: "waiting" field on input/output intermediate break(\_through) locations to respect services times [#3849](https://github.com/valhalla/valhalla/pull/3849)
* ADDED: --bbox & --geojson-dir options to valhalla_build_extract to only archive a subset of tiles [#3856](https://github.com/valhalla/valhalla/pull/3856)
* CHANGED: Replace unstable c++ geos API with a mix of geos' c api and boost::geometry for admin building [#3683](https://github.com/valhalla/valhalla/pull/3683)
* ADDED: optional write-access to traffic extract from GraphReader [#3876](https://github.com/valhalla/valhalla/pull/3876)
Expand Down
Loading

0 comments on commit 30e11f1

Please sign in to comment.