From f4c26d777b3f0901790a29cac64f2ab79d970e3e Mon Sep 17 00:00:00 2001 From: Colin Dawson Date: Tue, 3 Aug 2021 21:55:06 +0100 Subject: [PATCH] Added more logging around the tracking rates. Corrected the Command logged messages to use the correct info. Added a try catch to the setup dialog to see if an error occurs to give a better message. --- .../TelescopeUnitTests.cs | 6 +-- Meade.net.Telescope/Telescope.cs | 50 ++++++++++++------- Meade.net/SharedResources.cs | 33 +++++++----- 3 files changed, 57 insertions(+), 32 deletions(-) diff --git a/Meade.net.Telescope.UnitTests/TelescopeUnitTests.cs b/Meade.net.Telescope.UnitTests/TelescopeUnitTests.cs index 0b9a1d9..6ffcf43 100644 --- a/Meade.net.Telescope.UnitTests/TelescopeUnitTests.cs +++ b/Meade.net.Telescope.UnitTests/TelescopeUnitTests.cs @@ -822,9 +822,9 @@ namespace Meade.net.Telescope.UnitTests Assert.That(exception.Message, Is.EqualTo("Not connected to telescope when trying to execute: AlignmentMode Set")); } - [TestCase("AUTOSTAR", "43Eg", AlignmentModes.algAltAz, "AA")] - [TestCase("AUTOSTAR", "43Eg", AlignmentModes.algPolar, "AP")] - [TestCase("AUTOSTAR", "43Eg", AlignmentModes.algGermanPolar, "AP")] + [TestCase(TelescopeList.Autostar497, TelescopeList.Autostar497_43Eg, AlignmentModes.algAltAz, "AA")] + [TestCase(TelescopeList.Autostar497, TelescopeList.Autostar497_43Eg, AlignmentModes.algPolar, "AP")] + [TestCase(TelescopeList.Autostar497, TelescopeList.Autostar497_43Eg, AlignmentModes.algGermanPolar, "AP")] public void AlignmentMode_Set_WhenConnected_ThenSendsExpectedCommand(string productName, string firmware, AlignmentModes alignmentMode, string expectedCommand) { _sharedResourcesWrapperMock.Setup(x => x.ProductName).Returns(productName); diff --git a/Meade.net.Telescope/Telescope.cs b/Meade.net.Telescope/Telescope.cs index 2c8928e..339854a 100644 --- a/Meade.net.Telescope/Telescope.cs +++ b/Meade.net.Telescope/Telescope.cs @@ -339,7 +339,7 @@ namespace ASCOM.Meade.net public void CommandBlind(string command, bool raw) { - LogMessage("CommandBlind", "raw: {0} command {0}", raw, command); + LogMessage("CommandBlind", $"raw: {raw} command {command}"); CheckConnected("CommandBlind"); // Call CommandString and return as soon as it finishes //this.CommandString(command, raw); @@ -352,16 +352,16 @@ namespace ASCOM.Meade.net public bool CommandBool(string command, bool raw) { - LogMessage("CommandBool", "raw: {0} command {0}", raw, command); + LogMessage("CommandBool", $"raw: {raw} command {command}"); CheckConnected("CommandBool"); var result = SharedResourcesWrapper.SendBool(command, raw); - LogMessage("CommandBool", "Completed: {0}", result); + LogMessage("CommandBool", $"Completed: {result}"); return result; } public string CommandString(string command, bool raw) { - LogMessage("CommandString", "raw: {0} command {0}", raw, command); + LogMessage("CommandString", $"raw: {raw} command {command}"); CheckConnected("CommandString"); // it's a good idea to put all the low level communication with the device here, // then all communication calls this function @@ -377,7 +377,7 @@ namespace ASCOM.Meade.net { result = SharedResourcesWrapper.SendString(command, raw); } - LogMessage("CommandString", "Completed: {0}", result); + LogMessage("CommandString", $"Completed: {result}"); return result; } @@ -622,13 +622,26 @@ namespace ASCOM.Meade.net return false; } - private bool IsGWCommandSupported() => FirmwareIsGreaterThan(TelescopeList.Autostar497_43Eg); + private bool IsGwCommandSupported() + { + switch (SharedResourcesWrapper.ProductName) + { + case TelescopeList.LX200CLASSIC: + return false; + case TelescopeList.Autostar497: + return FirmwareIsGreaterThan(TelescopeList.Autostar497_43Eg); + case TelescopeList.LX200GPS: + return FirmwareIsGreaterThan(TelescopeList.LX200GPS_42G); + default: + return false; + } + } // true iff the mount will perform a meridian flip when required // According to "A User's Guide to the Meade LXD55 and LXD75 Telescopes" Autostar supports meridian flip so // we assume that for any telescope that supports the GW command and is not in Alt-Az mode then // meridian flip on slew is supported - private bool IsMeridianFlipOnSlewSupported() => IsGWCommandSupported() && AlignmentMode != AlignmentModes.algAltAz; + private bool IsMeridianFlipOnSlewSupported() => IsGwCommandSupported() && AlignmentMode != AlignmentModes.algAltAz; private bool FirmwareIsGreaterThan(string minVersion) { @@ -955,7 +968,7 @@ namespace ASCOM.Meade.net CheckConnected("AlignmentMode Get"); - if (IsGWCommandSupported()) + if (IsGwCommandSupported()) { var alignmentStatus = GetScopeAlignmentStatus(); return alignmentStatus.AlignmentMode; @@ -996,7 +1009,7 @@ namespace ASCOM.Meade.net CheckConnected("AlignmentMode Set"); //todo tidy this up into a better solution that means can :GW#, :AL#, :AA#, & :AP# and checked for Autostar properly - if (!IsGWCommandSupported()) + if (!IsGwCommandSupported()) throw new PropertyNotImplementedException("AlignmentMode", true); switch (value) @@ -1023,6 +1036,7 @@ namespace ASCOM.Meade.net private AlignmentStatus GetScopeAlignmentStatus() { + LogMessage("GetScopeAlignmentStatus", "Started"); var alignmentString = CommandString("GW", false); //:GW# Get Scope Alignment Status //Returns: # @@ -1045,6 +1059,7 @@ namespace ASCOM.Meade.net alignmentStatus.AlignmentMode = AlignmentModes.algGermanPolar; break; } + alignmentStatus.Tracking = alignmentString[1] == 'T'; switch (alignmentString[2]) { @@ -1068,6 +1083,7 @@ namespace ASCOM.Meade.net break; } + LogMessage("GetScopeAlignmentStatus", $"Result {alignmentStatus}"); return alignmentStatus; } @@ -1309,7 +1325,7 @@ namespace ASCOM.Meade.net { get { - var canSetTracking = IsGWCommandSupported(); + var canSetTracking = IsGwCommandSupported(); LogMessage("CanSetTracking", "Get - " + canSetTracking); return canSetTracking; } @@ -2591,7 +2607,7 @@ namespace ASCOM.Meade.net get { LogMessage("Tracking", "Get"); - if (IsGWCommandSupported()) + if (IsGwCommandSupported()) { var alignmentStatus = GetScopeAlignmentStatus(); return alignmentStatus.Tracking; @@ -2607,7 +2623,7 @@ namespace ASCOM.Meade.net } LogMessage("Tracking Set", $"{value}"); - SharedResourcesWrapper.SendBlind(value ? "AP" : "AL"); + SharedResourcesWrapper.SendBlind(value ? "AP" : "AL"); //todo need to route this to the real commands. } } @@ -2617,11 +2633,11 @@ namespace ASCOM.Meade.net { var rate = CommandString("GT", false); - if (rate == "+60.1") - return DriveRates.driveSidereal; + DriveRates result = rate == "+60.1" ? DriveRates.driveSidereal : DriveRates.driveLunar; - // we only support two rates ATM so return lunar tracking rate - return DriveRates.driveLunar; + LogMessage("TrackingRate Get", $"{rate} {result}"); + + return result; } set { @@ -2737,7 +2753,7 @@ namespace ASCOM.Meade.net return utcDate; } - catch (ParkedException e) + catch (ParkedException) { if (ParkedBehaviour == ParkedBehaviour.NoCoordinates) throw; diff --git a/Meade.net/SharedResources.cs b/Meade.net/SharedResources.cs index 975b093..2e51fdf 100644 --- a/Meade.net/SharedResources.cs +++ b/Meade.net/SharedResources.cs @@ -292,25 +292,34 @@ namespace ASCOM.Meade.net public static void SetupDialog() { - var profileProperties = ReadProfile(); - - using (SetupDialogForm f = new SetupDialogForm()) + try { - f.SetProfile(profileProperties); + var profileProperties = ReadProfile(); - if (IsConnected()) + using (SetupDialogForm f = new SetupDialogForm()) { - f.SetReadOnlyMode(); - } + f.SetProfile(profileProperties); - var result = f.ShowDialog(); - if (result == DialogResult.OK) - { - profileProperties = f.GetProfile(); + if (IsConnected()) + { + f.SetReadOnlyMode(); + } - WriteProfile(profileProperties); // Persist device configuration values to the ASCOM Profile store + var result = f.ShowDialog(); + if (result == DialogResult.OK) + { + profileProperties = f.GetProfile(); + + WriteProfile( + profileProperties); // Persist device configuration values to the ASCOM Profile store + } } } + catch (Exception e) + { + Console.WriteLine(e); + throw; + } } #endregion