-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Add comprehensive Features documentation #6966
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
base: dev-2.x
Are you sure you want to change the base?
Conversation
Add Features.md with complete overview of OTP capabilities organized into logical categories. Integrate the features page into documentation navigation and add cross-references from key documentation pages. Changes: - Add doc/user/Features.md with comprehensive feature list - Update mkdocs.yml to include Features in About section - Add links to Features.md from index, Product-Overview, Configuration, RoutingModes, and Basic-Tutorial pages - Update Transit Group Priority description
3450deb to
9d6cd1f
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## dev-2.x #6966 +/- ##
=============================================
- Coverage 72.14% 72.13% -0.01%
+ Complexity 19673 19672 -1
=============================================
Files 2127 2127
Lines 79562 79562
Branches 8041 8041
=============================================
- Hits 57397 57396 -1
- Misses 19331 19332 +1
Partials 2834 2834 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
| **Timetable Snapshot Management** | ||
| - Intelligent caching and throttling of real-time timetable snapshots to optimize performance. |
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.
Is that a feature or an implementation detail?
|
|
||
| **Backwards Delay Propagation** | ||
| - Configurable propagation of delays backwards through trip patterns when upstream data is missing. | ||
| [](Is this correct? Does this only apply to SIRI) |
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 applies to GTFS-RT.
| **Rate Limiting** | ||
| - Query complexity-based rate limiting for GraphQL APIs to prevent overload. |
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.
Hmm, I would not call that rate limiting.
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.
This should be removed, I missed it in my read-through.
| **Adaptive Pruning** | ||
| - Distance-based threshold adjustment for more aggressive pruning of problematic disconnected areas. |
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.
Part of island pruning.
| **Adaptive Pruning** | ||
| - Distance-based threshold adjustment for more aggressive pruning of problematic disconnected areas. | ||
|
|
||
| **Multi-threaded Graph Building** |
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.
Misleading title.
| - Option to cache OSM data in memory during processing instead of streaming multiple passes. | ||
|
|
||
| **Binary Trip Search** | ||
| - Optimized trip departure time lookup using binary search for better performance on high-frequency routes. |
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.
Seems like gibberish.
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.
It is not.
| **Binary Trip Search** | ||
| - Optimized trip departure time lookup using binary search for better performance on high-frequency routes. | ||
|
|
||
| **Search Thread Pool** |
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.
We have that, but does it work?
| **Time-restricted Access** | ||
| - Support for OSM opening hours on paths and streets that vary by time of day. |
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.
We don't have that.
| **GTFS Fares V2** | ||
| - Experimental support for the new Fares V2 specification including fare products and media. | ||
|
|
||
| **Interagency Fares** |
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.
It works across agencies and feeds but you cannot refer to any entities across feeds, which what many producers want.
| - Option to embed router configuration in the graph for fully self-contained deployments. | ||
|
|
||
| **Container Deployment** | ||
| - Docker/container-ready with health check endpoints and environment variable support. |
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.
No Docker healthchecks, you have to implement it yourself.
| **Build Reports** | ||
| - Detailed HTML reports of graph build process with errors, warnings, and statistics. | ||
|
|
||
| **Data Import Reports** | ||
| - Comprehensive validation reports for GTFS, NeTEx, and OSM data quality issues. |
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.
These are the same thing.
| - Mode-specific reluctance factors to prefer certain transit types over others. | ||
|
|
||
| **Walk Limits** | ||
| - Configurable maximum walk distances and times for access, egress, and transfers. |
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.
😳 ... maxWalkDistance...?
| **Ride Hailing** | ||
| - Integration with Uber API for real-time ride-hailing cost and time estimates. | ||
|
|
||
| **IBI Accessibility Score** | ||
| - Experimental accessibility scoring system for rating itinerary wheelchair accessibility (0-1 scale). | ||
|
|
||
| **Stop Consolidation** | ||
| - Merge nearby stops into logical groups for cleaner user interfaces and simpler trip planning. |
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.
I don't really want to advertise these more than we already do.
|
@leonardehrenfried I pushed this to |
|
Not a problem - let's discuss tomorrow. |
Summary
Addresses
Relates to documentation improvements for better feature discoverability
Documentation
✅ This PR is entirely documentation - adds a comprehensive features page to help users understand all OTP capabilities in one place
Changelog
🟥 Not relevant - documentation-only change
Bumping the serialization version id
🟥 No changes to serialized classes