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

doc: Change descriptions of keys for wind data to include standard identifiers such as AWS #590

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

chris0348
Copy link

The description of the sk-keys for wind data did not refer to the standard identifiers commonly used in watersports (e.g. AWS, AWA, TWD, ...). For an unambiguous interpretation of what the specific keys represent the standard identifiers are added to the description of the keys.

…entifiers such as AWS

The description of the sk-keys for wind data did not refer to the standard identifiers commonly used in watersports (e.g. AWS, AWA, TWD, ...). For an unambiguous interpretation of what the specific keys represent the standard identifiers are added to the description of the keys.
@@ -901,31 +901,32 @@ The angle the wind needs to shift to raise an alarm


## environment.wind.directionMagnetic
The wind direction relative to magnetic north
Wind direction based on speed through water (STW), relative to magnetic north
Copy link
Member

Choose a reason for hiding this comment

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

The wind direction relative to magnetic north is absolute - what you would experience being stationary, so definitely not based on STW.

Copy link
Author

@chris0348 chris0348 Nov 1, 2020

Choose a reason for hiding this comment

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

I intended to point out that "directionMagnetic" is referring to the wind over water and not to wind over ground, hence the reference to STW. But maybe misleading. What do you think of "Wind direction over water, relative to magnetic north"?


**Units:**rad


## environment.wind.directionTrue
The wind direction relative to true north
True Wind Direction (TWD) based on speed through water (STW), relative to true north
Copy link
Member

Choose a reason for hiding this comment

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

Again not based on STW.


**Units:**m/s


## environment.wind.speedOverGround
Wind speed over ground (as calculated from speedApparent and vessel's speed over ground)
Ground Wind Speed (GWS), calculated from apparent wind speed (AWS) and speed over ground (SOG)
Copy link
Member

Choose a reason for hiding this comment

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

speedApparent in the original is used specifically to refer to the Signal K path, I don't want to drop that. Also I'd like to keep the word vessel.

Copy link
Author

Choose a reason for hiding this comment

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

Ok, will revert to "speedApparent (AWS)" and "vessel's speed over ground"

@@ -303,27 +303,27 @@
"units": "rad"
},
"directionTrue": {
"description": "The wind direction relative to true north",
"description": "True Wind Direction (TWD) based on speed through water (STW), relative to true north",
Copy link
Member

Choose a reason for hiding this comment

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

sic

"$ref": "../definitions.json#/definitions/numberValue",
"units": "rad"
},
"directionMagnetic": {
"description": "The wind direction relative to magnetic north",
"description": "Wind direction based on speed through water (STW), relative to magnetic north",
Copy link
Member

Choose a reason for hiding this comment

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

sic

"$ref": "../definitions.json#/definitions/numberValue",
"units": "rad"
},
"speedTrue": {
"description": "Wind speed over water (as calculated from speedApparent and vessel's speed through water)",
"description": "True Wind Speed (TWS) over water, calculated from apparent wind speed (AWS) and speed through water (STW)",
Copy link
Member

Choose a reason for hiding this comment

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

sic

Copy link
Author

@chris0348 chris0348 Nov 1, 2020

Choose a reason for hiding this comment

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

Ok, will revert to "speedApparent (AWS)" and "vessel's speed trough water"

"$ref": "../definitions.json#/definitions/numberValue",
"units": "m/s"
},
"speedOverGround": {
"description": "Wind speed over ground (as calculated from speedApparent and vessel's speed over ground)",
"description": "Ground Wind Speed (GWS), calculated from apparent wind speed (AWS) and speed over ground (SOG)",
Copy link
Member

Choose a reason for hiding this comment

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

sic

Copy link
Author

Choose a reason for hiding this comment

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

Ok, will revert to "speedApparent (AWS)" and "vessel's speed over ground"

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.

2 participants