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

Add file & note to README; add new var to agency.txt; misc text cleanup throughout #57

Merged
merged 19 commits into from
Jun 12, 2018

Conversation

ddorinson
Copy link
Contributor

Two "structural" changes to the spec:

  1. README file did not mention fare_attributes.txt at all, even though we include it in the repository. Added this file to table of optional files, with footnote below to clarify.
  2. GTFS spec now has agency_email as an optional field in agency.txt; added this to GTFS-PLUS spec for consistency.

Plus various edits throughout to address:

  • Minor corrections to fare example tables for consistency with latest spec.
  • Cleaned up stray references to old files & fields that have been updated in latest spec.
  • Misc typos, spelling, capitalization, punctuation.
  • Added / corrected links to other files in the repo.
  • Added / corrected links to external data standards (e.g., currency codes).
  • Some markdown edits for cleaner table display.

ddorinson added 19 commits May 28, 2018 12:58
1. Corrected remaining instances of "fare_rules_ft.txt" to reference "fare_periods_ft.txt"
2. Corrected variable names cited in text descriptions from "origin_zone" to "origin_id" (same for "destination_")
3. Fixed file links that were pointing to wrong level in directory tree
4. Changes to content of example tables under "Systemwide Fare" and "Inter-agency Fare..." to match what seems to be intended from the text
This file contains all possible edits to handle inconsistencies flagged in osplanning-data-standards#54.  Once that is resolved, come back and review to confirm which changes stay/go.  Then merge into patch-2 branch for big pull request back to osplanning-data-standards repo.
Edits to address main repo Issue#54
@ddorinson
Copy link
Contributor Author

Hi @e-lo -- Just wanted to make sure you saw this pull request...? (I forgot to tag you when I first submitted.) Some of these changes are reflected in the final T2 memo, so please let me know if you see anything amiss to make sure I can fix it in the deliverable, too.

@e-lo e-lo merged commit c675f8c into osplanning-data-standards:master Jun 12, 2018
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.

2 participants