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

Bug: Correct processing of MWV message with "True" wind direction reference #99

Open
rgleason opened this issue Jan 28, 2024 · 10 comments

Comments

@rgleason
Copy link
Owner

rgleason commented Jan 28, 2024

Receiving MWV messages fixed with bdb PR. 27672af however now the values have to be fixed.

@rgleason
Copy link
Owner Author

rgleason commented Jan 28, 2024

Now we are getting data, but there are a couple of things that aren't working right.
Wind-T-A-True-Absolute

(Corrected) (Corrected again to use True Wind Direction

VDR file used to test this is Hakan's: VDR_Sensors-Hakan.txt

with Dashboard and the Imported Plugin for Windows Ov5.8.4
watchdog_pi-2.4.106.0-msvc-x86_64-wx32-10.0.22621-MSVC.tar.gz

I hope I have this answer right, haven't thought about it that much. This needs to show True wind
Is this coming from the instruments or is it calculated from Apparent wind and speed?
#99

Maybe there should be an option to calculate from from ship speed and apparent?
I am unclear about Watchdog protocols (can it derive or calculate a nmea0183 data?)
https://www.bwsailing.com/cc/2017/05/calculating-the-true-wind-and-why-it-matters/
True Wind Calculator https://www.starpath.com/calc/Weather%20Data%20Calculators/wind.html

Rick

@rgleason
Copy link
Owner Author

rgleason commented Jan 28, 2024

Dave, I got this totally wrong.
True Wind is the Direction from which the wind is blowing. Sorry.
Ignore the 180 in my screenshot. (Corrected the screenshot now)

@rgleason
Copy link
Owner Author

rgleason commented Jan 28, 2024

Closing Duplicate: Bug: Issue on version 2.4.106 (Wind T-A Alarm) -fixed? #74

@bdbcat
Copy link
Collaborator

bdbcat commented Jan 29, 2024

Please try again, simple words.

@rgleason
Copy link
Owner Author

rgleason commented Jan 29, 2024

Yes, you are right Dave it should become the value in Dashboard's "True Wind Direction", when Sync is clicked, but nothing happens sometimes, other times when there is a value in True Wind Direction, it puts something else in there.

Also I notice that sometimes if there is nothing in True Wind Direction, and Sync is clicked it calculates a direction based upon Wind Apparent Relative and SOG or STW? Do you think that the value calculated is correct?

I also find that the "Sync" button does not always work, it is locked out and nothing will change in the Value field. I don't know what causes this.

Hear are a couple of screenshots using
VDR_Sensors-Hakan.txt

Why is this one still frozen at 266 when I hit Sync and True Wind Direction is 188 in Dashboard? It does not make sense.

Wind-T-A-Corrected

Screenshot (1965)

@bdbcat
Copy link
Collaborator

bdbcat commented Jan 29, 2024

Sync button obviously broken.
I'll fix that first.

@rgleason
Copy link
Owner Author

Dave, I am attaching a Thomas Rauch file from the adriatic, which has MWV and is moving. It is an actual recording of Thomas vessel.
Tactics-sample2-5m.txt

Screenshot (1966)

@rgleason
Copy link
Owner Author

  1. Synch button is not working.
  2. When the "Value" is entered or selected as say "100" and the Range is "15" what displays is
    85 < 129 < 115
    100-15=85 Non-sesnical 100+15= 115
    Don't know what creates the middle number.

bdbcat added a commit to bdbcat/watchdog_pi that referenced this issue Jan 30, 2024
@bdbcat
Copy link
Collaborator

bdbcat commented Jan 30, 2024

PR posted to correct errors.

@rgleason
Copy link
Owner Author

#105

rgleason added a commit that referenced this issue Jan 30, 2024
Correct various math and logic errors in Wind Alarm  #99
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

No branches or pull requests

2 participants