Comments (12)
I've got accessibility_options changed to accessibility_attributes in all places in the spec with this commit. With review by @thekaveman we can close this issue.
@lastalfriday if you want to continue discussing the overlapping trips telemetry, which I think warrants discussion, please open a new issue and include your summary and screenshots!
from mobility-data-specification.
@schnuerle
Thank you for renaming to accessibility_attributes
. I see the changes live now in the release-2.0.0
branch.
However! I almost forgot this as we started discuss other things, my main point here was that this field should not be "Required", but either "Optional" or "Required if Available" (as you yourself suggested early on in this thread)
from mobility-data-specification.
Probably "Required if Available" is the way to go.
from mobility-data-specification.
@schnuerle how about for Trips? accessibility_options
is also listed as required there, should it be moved to "Required if Available"? https://github.com/openmobilityfoundation/mobility-data-specification/blob/release-2.0.0/data-types.md#trips
from mobility-data-specification.
Ok, I never really understood the idea behind "Required if available", but I'm guessing it has to do with encouraging providers to supply it if they have it..
Anyway, in the database it won't make a difference for me, it will still need to be "nullable".
"Naming things" is the hardest part of IT, so I have full understanding for this, but when you name something xxx_options
and it's not optional, well...
from mobility-data-specification.
"Naming things" is the hardest part of IT, so I have full understanding for this, but when you name something
xxx_options
and it's not optional, well...
Agreed on "naming things" being difficult!! Though "options" here is being used as a synonym for "choices" - unrelated to "optional" / "required".
from mobility-data-specification.
Ok, I get it. Confusing though :)
..could you not use "attributes" => accessibility_attributes
? ...since it's used other places..
I also found a couple of other properties that should not be required, but optional or "required if available".
It makes no sense for a required property to have the value null
I also see that this is correct for events, so..
from mobility-data-specification.
A (non-related) question regarding trip_ids
defined for telemetry:
I'm trying to make sense of this, but just can't...
In what scenario does a telemetry point belong to two trips ?
..and if there is such a scenario, why doesn't it apply to the journey as well?
from mobility-data-specification.
accessibility_options
is really meant to list the options available for accessibility. It doesn't have anything to do with if the field is optional! You can have accessibility options available or not, so listing them is optional.
But I do think accessibility_attributes
may be a better term here. I can make this change throughout.
For the side question of telemetry on multiple trips, the scenario came up in the working groups where the same trip in a car could be used to transport goods and/or people at the same time, under different billing and different customers. So trips can overlap within a mode or across multiple modes (like maybe food delivery at the same time as passenger delivery). It may apply for the journey as well, but I think this can roll up through the trips if needed, and trips have more attributes than journeys at the moment.
from mobility-data-specification.
Thanks for changing to accessibility_attributes
, that's much more clear
Ok, I guess that could be some kind of a scenario, even though I still have hard time seeing this as a real world scenario using mds, but that could ofc just be me being too deep in our own use cases.
..though I'm struggling to see how this many2one should be implemented...
Lets say we have: 2 trips that share 1 telemetry object like you describe
From the provider's perspective:
- They must first create 2 incomplete trip objects in their system to get
trip_ids
that will be needed to send ongoing telemetry (they're incomplete because the trips are not done) - These trip objects will only exist in their system (the trips are not done, so cannot send to the agency yet)
- They send telemetry to the agency with these trip_ids
- When the trips are done, they send the trips to the agency
From the agency's perspective:
- We receive telemetry points for trips that does not yet exist in our system
- We later receive these trips (if all goes well...)
Is this how it will work?
Even still... the "required" bit (explained with screenshots) should be changed to match events though?
from mobility-data-specification.
In reviewing Policy, I noticed there's a reference to Accessibility Options in the policy.rule
: https://github.com/openmobilityfoundation/mobility-data-specification/blob/release-2.0.0/policy/README.md#rules
In addition to changing the field name there, it might make sense to elevate this field out of Rule and up to the Policy document. Since the mode_id
is also at the Policy document level, and accessibility_attributes
depend on mode_id
.
from mobility-data-specification.
Thanks for the catch this is cleaned up here.
from mobility-data-specification.
Related Issues (20)
- Improve REST API docs HOT 2
- Agency POST response codes HOT 9
- Agency PUT Stops payload HOT 7
- Agency error and bulk responses HOT 11
- /vehicles in MDS 2.0 HOT 9
- MDS 2.0 - telemetry fields trip_ids/journey_id should correspond to the event HOT 2
- MDS 2.0 - trip/fare attributes inconsistencies HOT 7
- MDS Provider Id Registration HOT 1
- Getting vehicle event status in MDS 2.0 HOT 3
- Fix Reports - Register error table HOT 1
- MDS 2.0 - Stops should have same response structure as vehicles or vice versa HOT 4
- MDS 2.0 - round-trip car sharing and reservation/reserved HOT 4
- Correction of description vehicle events (car sharing) HOT 3
- Car_type as an vehicle attribute HOT 1
- Minor additions to vehicle attributes // Car sharing HOT 4
- Add geographies to be returned in Reports in Requirements HOT 2
- Reports endpoint - START_DATE clarification HOT 4
- guidance on how to use telemetry data for compliance HOT 3
- What vehicles are included in the MDS 2.0 /vehicles endpoint? HOT 4
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from mobility-data-specification.