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

Which version to use now? #44

Closed
rgleason opened this issue Jun 18, 2023 · 3 comments
Closed

Which version to use now? #44

rgleason opened this issue Jun 18, 2023 · 3 comments

Comments

@rgleason
Copy link
Owner

rgleason commented Jun 18, 2023

Watchdog v2.4.53.1 ( branch Update2) has this, but does not show a Pypilot Alarm, but does it have that?
Anchor
Speed
Course
Weather
Wind
Deadman
NMEA Data
Depth
Landfall
Boundary Time
Boundary Distance
Boundary Inclusion Alarm
The "New" alarm dialog box could be fixed again with wxFormBuilder. - don't have time really. This version seems to have the same alarms as the earlier version, and no Rudder or Pypilot Alarm.

Anyway, I am going to push this up to my remote branch:update2 as version 2.4.54

Watchdog-2 4 53 1



Watchdog 2.4.51 (branch: master) which we have been using has (no Pypilot Alarm)
Anchor
Speed
Course
Weather
Wind
Deadman
NMEA Data
Depth
Landfall
Boundary Time
Boundary Distance
Boundary Inclusion Alarm

The "New" alarm dialog box is bigger and works better.
The Rudder alarm does not appear to work..

Watchdog-2 4 51

I don't know if I should go back to the current version of watchdog 2.4.51 and simply update that for the time being.

If we decide on version 2.4.51 I will update remote branch:master TP1.0.238 and label it as 2.4.55

@rgleason
Copy link
Owner Author

rgleason commented Jun 18, 2023

I looked at the files for Watchdog v2.4.53.1 ( branch Update2)

and it does have pypilot alarm code in

pypilot_client.cpp
watchdogUI.h class pypilotPanel : public wxPanel
watchdogUI.cpp pypilotPanel::~pypilotPanel()
NewAlarmDialog.cpp // m_lAlarmType->InsertItem(PYPILOT, _("pypilot Alarm"));
// m_lAlarmType->InsertItem(RUDDER, _("Rudder"));
EditAlarmDialog.cpp // void pypilotPanel::OnAboutHardwareErrors( wxCommandEvent& )

I am not sure if this matches what Sean has, but I can check it when I get back.

LATER:
Watchdog v2.4.51 +TP1.0.22 ( branch master)
also has commented out pypilot code in similar places, but it has
this file signalK_client.cpp instead of pypilot_client.cpp

Sean's current version is using pypilot_client.cpp
Sean's is 2.4.39.0 + TP 1.0.185.0 fix macos from 2 years ago.

His most recent commit is Feb8 2022 Fix include location fix include location
before that on Jan 6 2022 he has fix issues with pypilot alarm.

@rgleason
Copy link
Owner Author

rgleason commented Jun 18, 2023

Going to push both versions to see if there are any problems with either one.

watchdog 2.4.55 + TP1.0.238 branch: master has pypilot code with signalk_client.cpp

watchdog 2.4.54 + fix bld isnan branch: update2 has pypilot code with pypilot_client.cpp (same as Sean's)

@rgleason
Copy link
Owner Author

rgleason commented Aug 6, 2023

The problems have been resolved by bdbcat. The plugin is no longer diverging and a PR has been made back to Sean's repos.
Everything is in the master branch now.

bdbcat July 30 commits for rudder, pypilot alarm, depth parsing
dd488d0
76524e3
5fad00e

@rgleason rgleason closed this as completed Aug 6, 2023
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

1 participant