mobility-data-specification
mobility-data-specification copied to clipboard
Improve description field in Policy to require more human readable details
New examples for Policy with agreed upon rule types - updated by Justin Yeo after discussion with Jean Kao
name: Improve description field description in Policy to require more human readable details about: Suggest changes to MDS title: Improve description field description in Policy to require more human readable details
MDS Pull Request
Thank you for your contribution! Please review our OMF contributing page to understand guidelines and policies for participation, and our Code of Conduct page.
To avoid complications and help make the Review process as smooth as possible, make sure to:
- Target
dev
branch. Please ensure you are targetingdev
, notmain
. - Keep the "Allow edits from maintainers" button checked to help us resolve some issues for you.
- Be ready to resolve any merge conflicts before we approve your Pull Request.
- Have an up to date profile, per our Github community profile guildance.
Explain pull request
Improve description field description in Policy to require more human readable details
Is this a breaking change
- No, not breaking
Impacted Spec
Which spec(s) will this pull request impact?
-
policy
Additional context
Issue from https://github.com/openmobilityfoundation/mobility-data-specification/issues/764 Decided upon in https://github.com/openmobilityfoundation/mobility-data-specification/wiki/Web-conference-notes,-2022.06.23-(MDS-Working-Group)
Thanks @jyeo17 ! Could you resolve the conflicts with policy/examples/README.md
and click here to sign the CLA: https://cla-assistant.io/openmobilityfoundation/mobility-data-specification?pullRequest=786
@schnuerle thanks for going through it! I have completed them. I am not sure who added the 3 examples at the bottom but I have left them there.
Currently the PR has these examples defined:
- Operating Area
- No Riding
- No Parking
- Parking
- Parking Time Limit
- Speed Limit
- Distribution Policies
- Tiered Parking Fees
- Provider Caps or Minimums
and in the planning doc we collectively listed these use cases:
- Operating Area
- No Riding
- No Parking
- Provider Caps or Minimums
- Distribution Policies
- Parking Time Limit
- Speed Limits
- Per Trip Fees
- Vehicle Right of Way Fees
- Metered Parking Fees
- Required Parking
- Preferred Parking
- Tiered Parking Fees
- Tiered Parking Fees Total
- Pick-up and Drop-off Fees
- Emergency Guidance
- Publish Event Areas
- Periodic maintenance enforcement
- Required parking centroid
- Registration fee
Which leaves these use cases not yet addressed.
- Per Trip Fees
- Vehicle Right of Way Fees
- Metered Parking Fees
- Tiered Parking Fees Total
- Pick-up and Drop-off Fees
- Emergency Guidance
- Publish Event Areas
- Periodic maintenance enforcement
- Required parking centroid
- Registration fee
And these two which I don't think are covered by just 'Parking':
- Required Parking (this might be what Parking is in the PR now)
- Preferred Parking
Do you think you could add these now? @jyeo17 @S-eb @jean-populus
If not we could do it in a future pull request, and I'd like to capture that in issue #764 as a comment.
Currently the PR has these examples defined:
- Operating Area
- No Riding
- No Parking
- Parking
- Parking Time Limit
- Speed Limit
- Distribution Policies
- Tiered Parking Fees
- Provider Caps or Minim
@schnuerle
The decision was taken at the task force level to tackle only the policies where 1- automation brings efficiency 2- policy rules are likely to change a lot
Therefore the short list for the following policies:
Operating Area
- No Riding
- No Parking
- Parking
- Parking Time Limit
- Speed Limit
- Distribution Policies
- Tiered Parking Fees
- Provider Caps or Minim
This has been validated by the WG steering committee.
The decision was also taken to not consider fees as part of the policy taskforce. the consensus being that fees is probably a different object that should be treated separately from policies.
Thanks
Seb
Just a note, Tiered Parking Fees was not part of the discussion and so has been removed.
We have:
- Operating Area
- No Riding
- No Parking
- Parking
- Parking Time Limit
- Speed Limit
- Distribution Policies
- Provider Caps or Minimums
After the discussion in the working group, I propose that we keep the definitions of the 2 main definitions of use cases that were discussed:
-
Operating area: The vehicle should stay within the areas of operation defined (Riding area). Reasoning: It would be the city's decision if they want to enforce such a policy or allow scooters to ride out of the city. If they allow so, no policy is required. If they want to enforce a boundary around the city, the operating area policy defines the riding area (except for no riding zones within).
-
No parking: The vehicle should not be parked in one of these defined areas in the statuses Available, Reserved and Non-operational. Reasoning: In the working group call there was a discussion over whether or not the word 'must' or 'should' should be used. I stand by 'should' as the user should still be able to end a trip outside of a parking bay, triggering a violation. Perhaps some cities might want to implement it so that vehicles cannot be locked unless in parking bays, but it seems like a rare decision made so far.
For each of the 8 examples, could you either 1) update the full JSON file examples that appear in this directory or 2) remove the link to the full file for each example and delete the JSON files in the directory?
cc @S-eb @jean-populus
looks good to me (once @schnuerle's ask is addressed)
Done @schnuerle @marie-x @jean-populus
Merging this to dev so we can consolidate all the work from task forces and other open PRs.