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

Reference for new run_events.txt spec #66

Draft
wants to merge 15 commits into
base: main
Choose a base branch
from

Conversation

skyqrose
Copy link

@skyqrose skyqrose commented Apr 1, 2024

Write up of #60 in a format that can be merged. I've gone with the event_sequence idea instead of run_event_id. Other than that there are minor text changes and some reformatting, but no semantic changes.

This is currently stacked on top of #61 . The important changes are in index.md below ## TODS-Specific File Definitions, or you can look directly at the rendered version

I've also rewritten all of the examples (which is why there are so many deleted files). Here's a rendered version of examples.md

Status as of 2024-04-11: I've finished the examples, and the run_events portion of this is complete (though more comments are of course welcome). This PR is now just waiting on discussion on #61 to resolve. I think we were going to vote on that one before voting on this one, but assuming both proposals are accepted, we may want to do just a single merge from this PR, since this includes everything from that PR and also fixes things up to be more consistent.

* Rebrands "ODS" to "TODS"
* Restructures data to use `_supplement` structure
* Updates canonical URLs to the GitHub repo
- Corrects routes_supplement.txt instead of routes.txt
- Standardizes Primary Key capitalization
- Clarifies stop_times_supplement.txt grammar
`_supplement` files rebranded as Supplement files
- Clarifies Experimental field description
- Adds overview information on Supplement file structure
- Updates evaluation criteria
- Adds example per @skyqrose
- Adds Implications & Guidance
Copy link

github-actions bot commented Apr 1, 2024

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.

None yet

2 participants