Skip to content

Clarification of GTFS Realtime occupancy descriptions#259

Merged
barbeau merged 10 commits intogoogle:masterfrom
MobilityData:clarify-occupancies
Feb 1, 2021
Merged

Clarification of GTFS Realtime occupancy descriptions#259
barbeau merged 10 commits intogoogle:masterfrom
MobilityData:clarify-occupancies

Conversation

@sccmcca
Copy link
Contributor

@sccmcca sccmcca commented Jan 7, 2021

Following the conversation in #212, this pull request aims to disambiguate the intended use of OccupancyStatus as describing the nominal status of passenger occupancy levels, and occupancy_percentage as describing passenger occupancy levels on a linear scale.

Summary:

  • Clarification to OccupancyStatus as indicating the nominal status of passenger occupancy; removal of "percentage" language to avoid confusion with occupancy_percentage.
  • Editorial clarification of occupancy percentage description.
  • Added line breaks to Caution statements for improved legibility.

Feedback is welcomed.

- Clarification to OccupancyStatus as indicating nominal statuses of passenger occupancy; removal of "percentage" language to avoid confusion with occupancy_percentage.
- Editorial clarification of occupancy percentage description.
- Added line breaks to **Caution** statements for improved legibility.
@google-cla google-cla bot added the cla: yes label Jan 7, 2021
@sccmcca sccmcca mentioned this pull request Jan 7, 2021
@stevenmwhite
Copy link
Contributor

These changes are helpful, particularly with the added details.

scmcca and others added 2 commits January 8, 2021 16:12
Co-authored-by: Sean Barbeau <sjbarbeau@gmail.com>
scmcca added 2 commits January 12, 2021 08:20
- "status" to "state" in description
- `VehiclePosition.occupancy_percentage` to `occupancy_percentage`
- succinct rework of description sentence
- "status" to "state" in description
- `VehiclePosition.occupancy_percentage` to `occupancy_percentage`
- succinct rework of description sentence
@skinkie
Copy link
Contributor

skinkie commented Jan 14, 2021

+1 OpenGeo

@abyrd
Copy link

abyrd commented Jan 15, 2021

It does seem like a good idea to replace the word "percentage" on fields that do not actually contain percentages. However, this PR replaces "percentage" with "allocation", which implies that someone allocated the seats, i.e. someone assigned them to a specific person or usage. The new word seems less clear than the one it replaces.

It also replaces the expression "degree of passenger occupancy" with "nominal state of passenger occupancy". I think I understand the intended meaning here, that this is a "name for the occupancy state" rather than a quantitative measure, but I suspect most readers will perceive it as opaque jargon. In common usage "nominal" means something exists in name only, has characteristics different than those implied by its name, or is very small. In economics it might also mean something like "unadjusted", and in engineering something like "within acceptable range".

@sccmcca
Copy link
Contributor Author

sccmcca commented Jan 15, 2021

@abyrd Good points.

Perhaps we can replace "allocation" with "number" and "amount" such that the spec reads:

The vehicle or carriage has a large allocation number of seats available. The allocation amount of free seats out of the total seats available to be considered large enough to fall into this category is determined at the discretion of the producer.

As for "nominal", perhaps we can remove it and simply rely on the word "state" such that the spec reads:

The nominal state of passenger occupancy for the vehicle or carriage.

Thoughts?

@sccmcca
Copy link
Contributor Author

sccmcca commented Jan 22, 2021

Hi everyone,

It seems like we've settled on the final proposal. As the proposal concerns an editorial clarification to a historical feature, I believe we can assume implementation from at least one GTFS-realtime producer and consumer. Also, we are well past the required 7-day discussion period as outlined in the specification amendment process. Feel free to voice any remaining feedback. To be clear, this proposal is not suggesting to make official the experimental OccupanyStatus field. Rather, it is seeking editorial clarification.

Formally, I'm calling for a vote on the official adoption of the editorial changes proposed in this PR. The vote will close on 2021-01-29 at 23:59:59 UTC.

@skinkie
Copy link
Contributor

skinkie commented Jan 22, 2021

+1 OpenGeo

@gcamp
Copy link
Contributor

gcamp commented Jan 25, 2021

+1 Transit

@stevenmwhite
Copy link
Contributor

+1 GMV Sync

@sccmcca
Copy link
Contributor Author

sccmcca commented Feb 1, 2021

The vote ended on 2021-01-29 23:59:59 UTC with a unanimous consensus of 3 yes votes from OpenGeo (@skinkie), Transit (@gcamp), and GMV Sync (@stevenmwhite). As per the specification amendment process, this proposal is accepted! Thanks to everyone involved.

@barbeau barbeau merged commit 729503b into google:master Feb 1, 2021
@barbeau barbeau deleted the clarify-occupancies branch February 1, 2021 15:59
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.

7 participants