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

Improve machine-usability of location_verify_method #129

Closed
j-d-b opened this issue Sep 2, 2020 · 5 comments · Fixed by #199
Closed

Improve machine-usability of location_verify_method #129

j-d-b opened this issue Sep 2, 2020 · 5 comments · Fixed by #199
Assignees
Labels
Needs discussion This issue needs clarification/additional discussion or is inactive

Comments

@j-d-b
Copy link
Collaborator

j-d-b commented Sep 2, 2020

Currently,location_verify_method, on the road_event_data_source in v3.0, is a text field with a very general description:

The method used to verify the accuracy of the location information

While there is an example included, there is too much flexibility in what ends up in the content of this field for it to be useful for machines. It seems, based on the example "Survey accurate GPS equipment accurate to 0.1 cm", that this is intended to be just a human-readable, text description of the method. However, there is valuable data (accuracy!) here which could be parsed by machines if structured differently.

I would like to hear from other data producers what they are using as a location_verify_method, as in the example there are two pieces of separate information:

  1. Method: GPS
  2. Accuracy: 0.1 cm

I think spatial accuracy especially would helpful to have as a number (units separate). This could instead be added alongside/instead of "estimated" or "verified" for start/end coords, as currently even "verified" doesn't given that much information, especially when location_verify_method is not provided (it is optional).

Curious other's thoughts as it seems there is more information that could be added to the road_event_data_source to benefit machines.

@davidcraig4300
Copy link

davidcraig4300 commented Sep 3, 2020 via email

@j-d-b
Copy link
Collaborator Author

j-d-b commented Sep 3, 2020

@davidcraig4300 the 0.1cm came from the example for a values of location_verify_method that has been in the spec since before v2.0.

I was not intending to discuss the level of accuracy required or desired, just the ability for the producer to specify the level of accuracy of their feed with a distance unit rather than as part of a longer, string text field.

@Mahsa-Ettefagh Mahsa-Ettefagh added the Needs discussion This issue needs clarification/additional discussion or is inactive label Nov 5, 2020
@CraigMoore-Sea
Copy link
Collaborator

This should align with any recommendations about location verification coming out of the Workers Present group.

@sergebeaudry
Copy link

Adding those field would be beneficial for the data user to judge how they are going to handle the data.

  • some equipment will provide high level precision as DavidGraig mentionned above. Usually a moving equipment will be able to provide a great accuracy with the caveat that it needs to be always powered. Not a problem on expensive devices.

  • When we go with standing device: a device that is moved to a location and power up, and stay still. . The precision is not at the 1m level due to millions of factors. and it is going to take a while to get to a 1m accuracy. Just take your phone, turn it on, open a map software and see the precision, not great.. start walking.. it will become better. Are those equipment can sample 24x7 at 1Mhz, yes, however battery live and cost will become an issue. As equipment provider we have to balance between the precision and the overall cost and form factor of the solution.
    Do we need to know at 1m level where are the work zone? No for many applications, Yes for others.

So providing a source and an accuracy element would help.

@j-d-b j-d-b linked a pull request Aug 26, 2021 that will close this issue
@j-d-b
Copy link
Collaborator Author

j-d-b commented Dec 16, 2021

Resolved in #199.

@j-d-b j-d-b closed this as completed Dec 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs discussion This issue needs clarification/additional discussion or is inactive
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants