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

Clarifying use of GNSS and Timestamp #385

Merged
merged 1 commit into from
Jan 4, 2022

Conversation

erikbosch
Copy link
Collaborator

This PR tries to improve two areas of VSS

GNSS

Previously we did not explicitly state that coordinates shall be based on WGS84, and the description on altitude was a bit ambiguous as it mentioned elevation. Now it clearly states that it is altitude over WGS84 reference ellipsoid.

(As I understand it altitude in WGS84 context refer to height over WGS84 reference ellipsoid, but elevation typically refers to height over WGS84 geoid (which varies depending on WGS84 revision used, e.g. EGM96, EGM2008)

Timestamp

We have recently started to specify that string formatted according to ISO 8601 shall be used for some signals. But ISO 8601 gives you a lot of alternatives and you do not even need to specify time zone, and then the string shall be interpreted as local time. We see no real reason to use anything else than UTC in signals, and we think it would be good to give additional details/recommendations in documentation.

The documentation proposal is reasonable aligned with what is specified by VISS and W3C:

https://www.w3.org/TR/NOTE-datetime
https://raw.githack.com/w3c/automotive/gh-pages/spec/VISSv2_Core.html#timestamp

-Clarifying that coordinates are based on WGS84
-Giving additional guidelines om timestamps, inspired by:

https://www.w3.org/TR/NOTE-datetime
https://raw.githack.com/w3c/automotive/gh-pages/spec/VISSv2_Core.html#timestamp
Copy link
Collaborator

@danielwilms danielwilms left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good, thanks

@danielwilms
Copy link
Collaborator

Decision 22/01/04: approved

@danielwilms danielwilms merged commit eb3e0d3 into COVESA:master Jan 4, 2022
@erikbosch erikbosch deleted the erik_gnss branch January 10, 2022 13:18
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.

None yet

2 participants