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

Human-readable names for destination layers in CSV output #929

Open
ansoncfit opened this issue Feb 1, 2024 · 1 comment
Open

Human-readable names for destination layers in CSV output #929

ansoncfit opened this issue Feb 1, 2024 · 1 comment
Labels

Comments

@ansoncfit
Copy link
Member

ansoncfit commented Feb 1, 2024

We have recently discussed making human-readable names (as opposed to our UUIDs) available in CSV outputs (see #909, #920). For these outputs, routing workers write complete rows as strings, which the broker then writes directly to CSVs in assemblers without further inspection/processing.

Workers have GTFS information readily available, so we can add route/stop names to the CSVs with the current design.

Workers do not have information about destination layers readily available other than the UUID. Names corresponding to these IDs are in the database, to which the workers are not connected. To write the names, we could:

  • Pass along a map from destination layer IDs to names in a snapshot file (similar to the scenario.json file)
  • Have the broker postprocess freeform access results it receives from workers, rather than simply writing the strings it receives directly to the CSV
  • Change our design so workers have database access

In the meantime, we should make add buttons to download the mappings from our IDs to names (for bundles and opportunity datasets) so users can more easily swap out the IDs for names themselves.

@abyrd
Copy link
Member

abyrd commented Jun 13, 2024

Discussing the first bullet point above: scenario.json is already a moment-in-time snapshot of some data from the database, but it's not per-regional-analysis (could be reused by many analyses). So it can't contain the id-to-destination map, which could be different for each regional analysis run.

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

No branches or pull requests

2 participants