GTFS Readiness Checklist
- Dominant language
- C++
- Stars
- 6.2k
- Forks
- 981
- Avg merge
- 2d 14h
- Merged PRs (30d)
- 14
Description
- [x] fix stitching in ingest_transit. for whatever reason the stitcher cant find the missing destination/origin for 1 of the stop pairs in the test. the result is we get a couple of logs that say we are bailing on one of the stop pairs. this is because the pair connects across the tile boundary (which is a great thing to test)
- [x] make sure convert transit can actually add the edges between stations (actually between stations platforms) so that you can actually route on a kRail or kBus edge
- [x] the routing algorithm doesnt work, says it cant find a transit station near by (shouldnt be the case we know the connection does work currently so perhaps). im hoping fixing the first 2 will make this one go away but i expect there to be more routing issues
- [x] fix the routing algorithms likely bugs due to the code being unexorcised for years
- IF WE MADE IT HERE WE COULD SAY WE ARE IN BETA WITH CAVEATS
- [x] hierarchy builder is not fully robust to the presence of transit data, so currently we cant build hierarchies if we want to have transit data as well. this in itself wouldnt stop us from supporting transit but its definitely required to be fixed before transit can be considered fully functional again.
- [x] unit test multi feed imports and fix if broken
- IF WE MADE IT HERE WE HAVE SOMETHING WE COULD SAY IS OUT OF BETA
- [ ] transit connect edges are not always formed on both sides of the street network edge, this happens because we currently dont look in adjacent tiles to find the end nodes. ive put a bunch of hints in the code about what to do to fix this (and also get better matches for edges to connect stations to)
- [ ] clean up the code in convert transit where we hook up the 3 transit node types and connect them to each other. i wrote a big todo in there
- [ ] another group of tasks to tackle adding transit to the graph after the graph is fully built as mentioned over in: https://github.com/valhalla/valhalla/issues/3806#issuecomment-1339231858
TODO: check off some of this list after we have merged #3988
Contributor guide
Assessment
This issue has not been assessed yet.