Skip to content
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

Enrich CSV output for detailed_itineraries() #400

Open
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

luyuliu
Copy link
Contributor

@luyuliu luyuliu commented Oct 3, 2024

Updated features:

  1. Add OSM link ID. Instead, the output network files via street_network__to_sf will output all links. See

    if (!edgeCursor.getFlag(EdgeStore.EdgeFlag.LINK)) {
    for more details. I removed the condition in the if clause since it would mess up the index system in the original road network. Now output as edge_id_list.

  2. Add boarding and alighting stop ID. Output as board_stop_id and alight_stop_id

  3. Since the LINSTRING contains both space and comma, I use tab as the delimitator for the csv file and semicolon as the delimitator for the edge_id_list

  4. Output time measures as second and distance measures as meters

luyuliu and others added 11 commits October 17, 2023 09:07
1. Seems like O/D with stop_id can be retrieved but don't know if that's index or actual stop_id

2. Weird error when trying to make a list of StreetSegment edgeID when walking. Try to access it by access the public variable edgeId but won't let me. The code should be correct. See test.java for more testing. But could be because of weird cross-package access limitation for java

Manage to make the jar work by using a try catch phrase,
… and board stop id is an index, not actual id.
… to accommodate the new update to java 21. didn't succeed.
@luyuliu
Copy link
Contributor Author

luyuliu commented Oct 3, 2024

Don't know which branch to merge to so I just select main which can be problematic. Just FYI.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant