Skip to content

docs: clarify v2.1.0 conformance requirements#156

Open
ChelseaKR wants to merge 2 commits into
MobilityData:mainfrom
ChelseaKR:agent/clarify-v210-conformance
Open

docs: clarify v2.1.0 conformance requirements#156
ChelseaKR wants to merge 2 commits into
MobilityData:mainfrom
ChelseaKR:agent/clarify-v210-conformance

Conversation

@ChelseaKR

@ChelseaKR ChelseaKR commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Summary

  • remove alignment padding from CSV examples so copied values do not contain unintended spaces
  • use the defined start_mid_trip and end_mid_trip field names throughout the run-events prose
  • spell out all four employee_run_dates.txt primary-key fields
  • state which GTFS Required fields an added Supplement row must provide
  • scope the conditional vehicle_assignments.service_id requirement to each row's block_id

Issue #152 disposition

This implements each clarification confirmed in #152:

  1. Example whitespace: removes the padding from every affected CSV example. Spaces that are part of a field value, such as Report Time, are unchanged.
  2. Mid-trip field names: replaces mid_trip_start and mid_trip_end in prose with the existing table fields, start_mid_trip and end_mid_trip.
  3. Employee assignment primary key: replaces Primary Key: * with (date, service_id, run_id, employee_id), matching GTFS's meaning of * while making the constraint explicit.
  4. Added Supplement rows: states that a row being added must define all fields marked Required in the corresponding GTFS file, plus any Conditionally Required field whose condition applies to that row. Updates and deletes still need only the Primary Key fields.
  5. Vehicle assignment service: makes service_id conditionally required only for a row whose block_id occurs under more than one service_id.

Scope

This PR does not change the Time definition. That clarification was handled in #150 and is already on main.

The changes clarify the intended behavior discussed in #152; they do not add new files or fields.

Validation

  • make build
  • git diff --check

Closes #152.

… word

Added Supplement rows must also satisfy any Conditionally Required GTFS
field whose condition applies, per the discussion in MobilityData#152. Also removes a
pre-existing doubled 'the' in the start_time and end_time descriptions.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

@skyqrose skyqrose left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM and satisfies all the discussion from #152

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.

Clarify remaining v2.1.0 field and conformance ambiguities

2 participants