V1.8.52 - Updates

- Moved LCD updates out of the mount loop.
- Fixed a bug that did not correctly handle displaying Roll and Pitch offset.
- Added ability to define DEC limits. Limits are not enforced yet, that code is still in development.
- Added new debug channel for digital level.

Author: ClutchplateDude

Software/Arduino code/OpenAstroTracker/Configuration_adv.hpp

@@ -209,7 +209,7 @@
 //
 // SETTINGS
 //
-  #define INFRA_SSID "YouSSID"
+  #define INFRA_SSID "YourSSID"
   #define INFRA_WPAKEY "YourWPAKey"
   #define OAT_WPAKEY "superSecret"
   #define HOSTNAME "OATerScope"
@@ -244,6 +244,7 @@
 #define DEBUG_VERBOSE        0x0080
 #define DEBUG_STEPPERS       0x0100
 #define DEBUG_EEPROM         0x0200
+#define DEBUG_GYRO           0x0400
 #define DEBUG_ANY            0xFFFF
 
 ////////////////////////////

Software/Arduino code/OpenAstroTracker/Version.h

@@ -1 +1 @@
-#define VERSION "V1.8.51"
+#define VERSION "V1.8.52"

Software/Arduino code/OpenAstroTracker/src/Mount.cpp

@@ -78,6 +78,7 @@
 
 // Extended bits
 #define EEPROM_PARKING_POS_MARKER_BIT         0x0001
+#define EEPROM_DEC_LIMIT_MARKER_BIT           0x0002
 
 const char* formatStringsDEC[] = {
   "",
@@ -155,6 +156,8 @@ Mount::Mount(int stepsPerRADegree, int stepsPerDECDegree, LcdMenu* lcdMenu) {
   _slewingToPark = false;
   _raParkingPos  = 0;
   _decParkingPos = 0;
+  _decLowerLimit = 0;
+  _decUpperLimit = 0;
 
   #if USE_GYRO_LEVEL == 1
   _pitchCalibrationAngle = 0;
@@ -324,6 +327,15 @@ void Mount::readPersistentData()
     else{
       LOGV1(DEBUG_INFO|DEBUG_EEPROM,F("Mount: EEPROM: No stored value for Parking position"));
     }
+    if (nextMarker & EEPROM_DEC_LIMIT_MARKER_BIT){
+      _decLowerLimit = EPROMStore::readInt32(31); // 31-34
+      _decUpperLimit = EPROMStore::readInt32(35); // 35-38
+      LOGV3(DEBUG_INFO|DEBUG_EEPROM,F("Mount: EEPROM: DEC limitsread as %l -> %l"), _decLowerLimit, _decUpperLimit );
+    }
+    else{
+      LOGV1(DEBUG_INFO|DEBUG_EEPROM,F("Mount: EEPROM: No stored value for Parking position"));
+    }
+    
   } 
   else {
     LOGV1(DEBUG_INFO|DEBUG_EEPROM,F("Mount: EEPROM: No ExtendedMarker present"));
@@ -444,7 +456,7 @@ void Mount::writePersistentData(int which, long val)
     case EEPROM_RA_PARKING_POS:
     case EEPROM_DEC_PARKING_POS:
     {
-      // ... set bit 8 to indicate pitch offset angle value has been written to 19/20
+      // ... set bit 0 in extended flag to indicate Parking pos has been written to 23-30
       writeExtended = true;
       extendedFlag |= EEPROM_PARKING_POS_MARKER_BIT;
       if (which == EEPROM_RA_PARKING_POS ){
@@ -457,6 +469,23 @@ void Mount::writePersistentData(int which, long val)
       }
     }
     break;
+
+    case EEPROM_DEC_UPPER_LIMIT:
+    case EEPROM_DEC_LOWER_LIMIT:
+    {
+      // ... set bit 1 in extended flag to indicate Parking pos has been written to 23-30
+      writeExtended = true;
+      extendedFlag |= EEPROM_DEC_LIMIT_MARKER_BIT;
+      if (which == EEPROM_DEC_UPPER_LIMIT ){
+        EPROMStore::updateInt32(31, val);
+        LOGV2(DEBUG_INFO|DEBUG_EEPROM,F("Mount: EEPROM Write: Updating DEC Upper limit to %l at 31-34"), val);
+      }
+      else {
+        EPROMStore::updateInt32(35, val);
+        LOGV2(DEBUG_INFO|DEBUG_EEPROM,F("Mount: EEPROM Write: Updating DEC Lower limit to %l at 35-38"), val);
+      }
+    }
+    break;
   }
 
 
@@ -704,6 +733,7 @@ float Mount::getPitchCalibrationAngle()
 void Mount::setPitchCalibrationAngle(float angle)
 {
     uint16_t angleValue = (angle * 100) + 16384;
+    LOGV3(DEBUG_GYRO, "Mount: Setting Pitch calibration to %d (%f)", angleValue, angle);
     writePersistentData(EEPROM_PITCH_OFFSET, angleValue);
     _pitchCalibrationAngle = angle;
 }
@@ -727,6 +757,7 @@ float Mount::getRollCalibrationAngle()
 void Mount::setRollCalibrationAngle(float angle)
 {
     uint16_t angleValue = (angle * 100) + 16384;
+    LOGV3(DEBUG_GYRO, "Mount: Setting Roll calibration to %d (%f)", angleValue, angle);
     writePersistentData(EEPROM_ROLL_OFFSET, angleValue);
     _rollCalibrationAngle = angle;
 }
@@ -1867,7 +1898,6 @@ void Mount::interruptLoop()
 // Process any stepper changes. 
 /////////////////////////////////
 void Mount::loop() {
-  unsigned long now = millis();
   bool raStillRunning = false;
   bool decStillRunning = false;
 
@@ -2005,7 +2035,7 @@ void Mount::loop() {
             _stepperDEC->moveTo(_decParkingPos);
             _totalDECMove = 1.0f * _stepperDEC->distanceToGo();
             _totalRAMove = 1.0f * _stepperRA->distanceToGo();
-            LOGV5(DEBUG_MOUNT|DEBUG_STEPPERS,F("Mount::Loop:   Park Position is R:%l  D:%l, TotalMove is R:%f, D:%f"), _raParkingPos,_decParkingPos,_totalRAMove, _totalDECMove);
+            LOGV5(DEBUG_MOUNT|DEBUG_STEPPERS,F("Mount::Loop:   Park Position is R:%l  D:%l, TotalMove is R:%f, D:%f"), _raParkingPos, _decParkingPos,_totalRAMove, _totalDECMove);
             if ((_stepperDEC->distanceToGo() != 0) || (_stepperRA->distanceToGo() != 0)) {
               _mountStatus |= STATUS_PARKING_POS | STATUS_SLEWING;
             }
@@ -2026,17 +2056,8 @@ void Mount::loop() {
 
         // Make sure we do one last update when the steppers have stopped.
         displayStepperPosition();
-        if (!inSerialControl) {
-          _lcdMenu->updateDisplay();
-        }
       }
     }
-
-    if ((_bootComplete) && (now - _lastTrackingPrint > 200)) {
-      _lcdMenu->printAt(15,0, isSlewingTRK() ? '&' : '`');
-      _lastTrackingPrint = now;
-    }
-
   }
 
   _stepperWasRunning = raStillRunning || decStillRunning;
@@ -2051,12 +2072,18 @@ void Mount::bootComplete() {
     _bootComplete = true;
 }
 
+bool Mount::isBootComplete(){
+  return _bootComplete;
+}
+
+
+
 /////////////////////////////////
 //
 // setParkingPosition
 //
 /////////////////////////////////
-void Mount::setParkingPosition(){
+void Mount::setParkingPosition() {
   _raParkingPos = _stepperRA->currentPosition() - _stepperTRK->currentPosition();
   _decParkingPos = _stepperDEC->currentPosition();
 
@@ -2066,6 +2093,42 @@ void Mount::setParkingPosition(){
   writePersistentData(EEPROM_DEC_PARKING_POS, _decParkingPos);
 }
 
+/////////////////////////////////
+//
+// setDecLimitPosition
+//
+/////////////////////////////////
+void Mount::setDecLimitPosition(bool upper) {
+  if (upper) {
+    _decUpperLimit = _stepperDEC->currentPosition();
+    writePersistentData(EEPROM_DEC_UPPER_LIMIT, _decUpperLimit);
+    LOGV3(DEBUG_MOUNT,F("Mount::setDecLimitPosition(Upper): limit DEC: %l -> %l"), _decLowerLimit, _decUpperLimit);
+  }
+  else{
+    _decLowerLimit = _stepperDEC->currentPosition();
+    writePersistentData(EEPROM_DEC_LOWER_LIMIT, _decLowerLimit);
+    LOGV3(DEBUG_MOUNT,F("Mount::setDecLimitPosition(Lower): limit DEC: %l -> %l"), _decLowerLimit, _decUpperLimit);
+  }
+}
+
+/////////////////////////////////
+//
+// clearDecLimitPosition
+//
+/////////////////////////////////
+void Mount::clearDecLimitPosition(bool upper) {
+  if (upper) {
+    _decUpperLimit = 0;
+    writePersistentData(EEPROM_DEC_UPPER_LIMIT, _decUpperLimit);
+    LOGV3(DEBUG_MOUNT,F("Mount::clearDecLimitPosition(Upper): limit DEC: %l -> %l"), _decLowerLimit, _decUpperLimit);
+  }
+  else{
+    _decLowerLimit = 0;
+    writePersistentData(EEPROM_DEC_LOWER_LIMIT, _decLowerLimit);
+    LOGV3(DEBUG_MOUNT,F("Mount::clearDecLimitPosition(Lower): limit DEC: %l -> %l"), _decLowerLimit, _decUpperLimit);
+  }
+}
+
 /////////////////////////////////
 //
 // setHome

Software/Arduino code/OpenAstroTracker/src/Mount.hpp

@@ -53,6 +53,8 @@
 #define EEPROM_ROLL_OFFSET 8
 #define EEPROM_RA_PARKING_POS 9
 #define EEPROM_DEC_PARKING_POS 10
+#define EEPROM_DEC_LOWER_LIMIT 11
+#define EEPROM_DEC_UPPER_LIMIT 12
 
 //////////////////////////////////////////////////////////////////
 //
@@ -206,6 +208,12 @@ class Mount {
   // Set the current stepper positions to be parking position.
   void setParkingPosition();
 
+  // Set the DEC limit position to the current stepper position. If upper is true, sets the upper limit, else the lower limit.
+  void setDecLimitPosition(bool upper); 
+  
+  // Clear the DEC limit position. If upper is true, clears upper limit, else the lower limit.
+  void clearDecLimitPosition(bool upper);
+
   // Auto Home with TMC2209 UART
   #if (RA_DRIVER_TYPE == DRIVER_TYPE_TMC2209_UART) || (DEC_DRIVER_TYPE == DRIVER_TYPE_TMC2209_UART)
     void startFindingHomeRA();
@@ -261,7 +269,7 @@ class Mount {
   int getBacklashCorrection();
 
   // Called when startup is complete and the mount needs to start updating steppers.
-  void  startTimerInterrupts();
+  void startTimerInterrupts();
 
   // Read the saved configuration from persistent storage
   void readConfiguration();
@@ -272,6 +280,9 @@ class Mount {
   // Get Mount configuration data
   String getMountHardwareInfo();
 
+  // Returns a flag indicating whether the mount is fully booted.
+  bool isBootComplete();
+
   // Let the mount know that the system has finished booting
   void bootComplete();
 private:
@@ -310,6 +321,8 @@ class Mount {
   int _moveRate;
   long _raParkingPos;
   long _decParkingPos;
+  long _decLowerLimit;
+  long _decUpperLimit;
 
 #if USE_GYRO_LEVEL == 1
   float _pitchCalibrationAngle;

Software/Arduino code/OpenAstroTracker/src/a_inits.hpp

@@ -113,6 +113,7 @@ bool inStartup = false;        // Start with a guided startup
 
 // Serial control variables
 bool inSerialControl = false; // When the serial port is in control
+bool okToUpdateMenu = true;   // Can be used to supress rendering the first line of the menu.
 bool quitSerialOnNextButtonRelease = false; // Used to detect SELECT button to quit Serial mode.
 
 // Global variables

Software/Arduino code/OpenAstroTracker/src/c65_startup.hpp

@@ -39,6 +39,7 @@ void startupIsCompleted() {
 
   startupState = StartupCompleted;
   inStartup = false;
+  okToUpdateMenu = true;
 
   mount.startSlewing(TRACKING);
 
@@ -87,6 +88,7 @@ bool processStartupKeys() {
             #else
             startupState = StartupWaitForPoleCompletion;
             inStartup = false;
+            okToUpdateMenu = false;
             lcdMenu.setCursor(0, 0);
             lcdMenu.printMenu("Home with ^~<>");
             lcdMenu.setActive(Control_Menu);

Software/Arduino code/OpenAstroTracker/src/c75_menuCTRL.hpp

@@ -121,15 +121,11 @@ bool processControlKeys()
           mount.startSlewing(TRACKING);
         }
 
-        // Set flag to prevent resetting zero point when moving over the menu items
-        ctrlState = HIGHLIGHT_MANUAL;
-
 #if SUPPORT_GUIDED_STARTUP == 1
         if (startupState == StartupWaitForPoleCompletion)
         {
           startupState = StartupPoleConfirmed;
           inStartup = true;
-          ctrlState = HIGHLIGHT_MANUAL;
         }
         else
 #endif
@@ -138,6 +134,7 @@ bool processControlKeys()
         }
 
         ctrlState = HIGHLIGHT_MANUAL;
+        okToUpdateMenu = true;
         setZeroPoint = true;
       }
       else if (key == btnLEFT)
@@ -192,6 +189,7 @@ bool processControlKeys()
         else
 #endif
         {
+          okToUpdateMenu = false;
           lcdMenu.setCursor(0, 0);
           lcdMenu.printMenu("Set home pos?");
           ctrlState = MANUAL_CONTROL_CONFIRM_HOME;