tom-r / tactics_pi

a performance enhancement of dashboard_pi for OpenCPN
16 stars 14 forks source link

changes in calculate true wind #45

Closed pa2wlt closed 1 year ago

pa2wlt commented 5 years ago

Hi Thomas, I changed CalculateTrue wind. TWS and TWA are now also being calculated without heading. For TWD still the heading is used. Only if there is no TWD available, the (less accurate) COG is used. I tested it thoroughly with my stream and that worked all fine. Please have a look at the changes. Kindest regards.

rgleason commented 4 years ago

Petri, do you think this should be added or not? I think Hakan made some similar changes to dashboard, but I should probably confirm that.

pa2wlt commented 2 years ago

@rgleason I see the pull request is still open. Anything I can do?

rgleason commented 2 years ago

If I know this was the right thing to do, I would accept a PR. BTW I have some other things that need to be done with Tactics in the next few days, So let me ask Hakan about your PR. first. And I should try to get the preferences fixed for Tactics for armhf builds.