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

Car share event types #843

Closed
thekaveman opened this issue Apr 23, 2023 · 8 comments
Closed

Car share event types #843

thekaveman opened this issue Apr 23, 2023 · 8 comments
Labels
Car Share Car share mode: cars, vehicles, short or long term rentals question Further information is requested Schema Implications for JSON Schema or OpenAPI State Machine Changes in the vehicle state events and state machine diagram
Milestone

Comments

@thekaveman
Copy link
Collaborator

Cancellation types issue

In the car-share mode, under Event Types, we have the following cancellation types:

  • driver_cancellation
  • passenger_cancellation
  • provider_cancellation

In the Trip ID Requirements we see these cancellation types:

  • driver_cancellation
  • customer_cancellation
  • provider_cancellation

Which one is correct (or are both?) for car-share:

  • customer_cancellation
  • passenger_cancellation

Trip types issue

Trip ID Requirements lists trip_pause and trip_resume.

Event Types only lists trip_resume, omitting trip_pause

Presumably trip_pause should be an allowed Event Type here.

@thekaveman thekaveman added the question Further information is requested label Apr 23, 2023
@schnuerle
Copy link
Member

I think passenger_cancellation is wrong for car share and only applies to passenger services. This should be fixed now for 2.0.

Yes and trip_pause should also be added to possible events.

@schnuerle schnuerle transferred this issue from openmobilityfoundation/mds-openapi Apr 24, 2023
@schnuerle schnuerle added this to the 2.0.0 milestone Apr 24, 2023
@schnuerle schnuerle added State Machine Changes in the vehicle state events and state machine diagram Car Share Car share mode: cars, vehicles, short or long term rentals Schema Implications for JSON Schema or OpenAPI labels Apr 24, 2023
@thekaveman
Copy link
Collaborator Author

I think passenger_cancellation is wrong for car share and only applies to passenger services.

Got it, so the correct list for car share is:

  • driver_cancellation
  • customer_cancellation
  • provider_cancellation

Yes and trip_pause should also be added to possible events.

Done.

@thekaveman
Copy link
Collaborator Author

This is being handled in OpenAPI in openmobilityfoundation/mds-openapi#6

@schnuerle
Copy link
Member

@schnuerle
Copy link
Member

Updated car share to change references to passenger to customer instead in this commit, and updated the state machine diagram here. Ready to close issue on confirmation by @thekaveman.

@thekaveman
Copy link
Collaborator Author

@schnuerle I think we still need this update:

trip_pause should also be added to possible events.

@schnuerle
Copy link
Member

schnuerle commented May 2, 2023

Looking at it I actually think trip_pause should not be in Car Share. The 4 listed should work fine. I think it needs to be removed from the Trip ID Requirements list here instead: https://github.com/openmobilityfoundation/mobility-data-specification/blob/release-2.0.0/modes/car-share.md#trip-id-requirements

Car Share

•	trip_end
•	trip_resume
•	trip_start
•	trip_stop	

Micro

•	trip_cancel
•	trip_end
•	trip_start

Robots

•	trip_end
•	trip_resume
•	trip_start
•	trip_pause

Passenger

•	trip_end
•	trip_resume
•	trip_start
•	trip_stop

All Available

Field Description
trip_cancel Cancel trip
trip_end End trip
trip_enter_jurisdiction Trip enters a jurisdiction
trip_leave_jurisdiction Trip leaves a jurisdiction
trip_pause Pause trip temporarily but do not end trip
trip_resume Resume trip
trip_start Start trip
trip_stop Stop trip

@schnuerle
Copy link
Member

Removed trip_pause from car share here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Car Share Car share mode: cars, vehicles, short or long term rentals question Further information is requested Schema Implications for JSON Schema or OpenAPI State Machine Changes in the vehicle state events and state machine diagram
Projects
None yet
Development

No branches or pull requests

2 participants