V1.6.25 - Updates

- Changed CTRL menu (and thus Guided startup Control) to move the steppers only while a button is depressed. This allows for much finer control and also is not as flaky as the other mode was. It also means movement in only one direction at a time is possible.

Author: ClutchplateDude

Software/Arduino code/OpenAstroTracker/DayTime.cpp

@@ -1,5 +1,9 @@
 #include "Utility.h"
 #include "DayTime.hpp"
+
+///////////////////////////////////
+// DayTime (and DegreeTime below)
+//
 // A class to handle hours, minutes, seconds in a unified manner, allowing
 // addition of hours, minutes, seconds, other times and conversion to string.
 

Software/Arduino code/OpenAstroTracker/OpenAstroTracker.ino

@@ -1,7 +1,7 @@
 /*
   =======================================================================================================================================
 
-    Version 1.6.22
+    Version 1.6
 
     1. Connect your Arduino, under tools choose "Arduino Uno", set the right Port and set "Arduino ISP" as the Programmer.
     2. Hit upload (Ctrl-U)
@@ -16,7 +16,7 @@
 */
 #include "Globals.h"
 
-String version = "V1.6.24";
+String version = "V1.6.25";
 
 ///////////////////////////////////////////////////////////////////////////
 // Please see the Globals.h file for configuration of the firmware.
@@ -56,13 +56,15 @@ int DECacceleration = 400;
 
 // Define some stepper limits to prevent physical damage to the tracker. This assumes that the home
 // point (zero point) has been correctly set to be pointing at the celestial pole.
+// Note: these are currently not used
 float RAStepperLimit = 15500;         // Going much more than this each direction will make the ring fall off the bearings.
 
 // These are for 47N, so they will need adjustment if you're a lot away from that.
 // You can use the CTRL menu to find the limits and place them here. I moved it
 // down until my lens was horizontal. Note the DEC number. Then move it up until
 // the lens is horizontal and note that number. Put those here. Always watch your
 // tracker and hit RESET if it approaches a dangerous area.
+// Note: these are currently not used
 float DECStepperDownLimit = 10000;    // Going much more than this will make the lens collide with the ring
 float DECStepperUpLimit = -22000;     // Going much more than this is going below the horizon.
 

Software/Arduino code/OpenAstroTracker/c75_menuCTRL.ino

@@ -2,6 +2,35 @@
 bool confirmZeroPoint = false;
 bool setZeroPoint = true;
 
+
+#define LOOPS_TO_CONFIRM_KEY 10
+byte loopsWithKeyPressed = 0;
+byte keyPressed = btnNONE;
+
+
+bool processKeyStateChanges(int key, int dir)
+{
+  bool ret = false;
+  if (keyPressed != key) {
+    loopsWithKeyPressed = 0;
+    keyPressed = key;
+  }
+  else {
+    if (loopsWithKeyPressed == LOOPS_TO_CONFIRM_KEY) {
+      mount.stopSlewing(ALL_DIRECTIONS);
+      mount.waitUntilStopped(ALL_DIRECTIONS);
+      if (dir != 0) {
+        mount.startSlewing(dir);
+      }
+      loopsWithKeyPressed++;      
+      ret = true;
+    }
+    else if (loopsWithKeyPressed < LOOPS_TO_CONFIRM_KEY) {
+      loopsWithKeyPressed++;
+    }
+  }
+}
+
 bool processControlKeys() {
   byte key;
 
@@ -35,8 +64,8 @@ bool processControlKeys() {
           startupState = StartupPoleConfirmed;
           inStartup = true;
         }
-        else 
-#endif        
+        else
+#endif
         {
           lcdMenu.setNextActive();
         }
@@ -50,56 +79,49 @@ bool processControlKeys() {
     return true;
   }
 
-  if (lcdButtons.keyChanged(key)) {
-    switch (key) {
-      case btnUP: {
-          if (!mount.isSlewingDEC()) {
-            mount.startSlewing(NORTH);
-          } else {
-            mount.stopSlewing(NORTH | SOUTH);
-          }
-        }
-        break;
-
-      case btnDOWN: {
-          if (!mount.isSlewingDEC()) {
-            mount.startSlewing(SOUTH);
-          } else {
-            mount.stopSlewing(NORTH | SOUTH);
-          }
-        }
-        break;
-
-      case btnLEFT: {
-          if (!mount.isSlewingRA()) {
-            mount.startSlewing(WEST);
-          } else {
-            mount.stopSlewing(EAST | WEST);
-          }
-        }
-        break;
-
-      case btnRIGHT: {
-          if (!mount.isSlewingRA()) {
-            mount.startSlewing(EAST);
-          } else {
-            mount.stopSlewing(EAST | WEST);
-          }
-        }
-        break;
+  mount.loop();
+
+  key = lcdButtons.currentState();
+  switch (key) {
+    case btnUP: {
+        processKeyStateChanges(btnUP, NORTH);
+      }
+      break;
+
+    case btnDOWN: {
+        processKeyStateChanges(btnDOWN, SOUTH);
+      }
+      break;
+
+    case btnLEFT: {
+        processKeyStateChanges(btnLEFT, WEST);
+      }
+      break;
 
-      case btnSELECT: {
-          mount.stopSlewing(ALL_DIRECTIONS);
-          mount.waitUntilStopped(ALL_DIRECTIONS);
+    case btnRIGHT: {
+        processKeyStateChanges(btnRIGHT, EAST);
+      }
+      break;
+
+    case btnSELECT: {
+        if (processKeyStateChanges(btnSELECT, 0))
+        {
           lcdMenu.setCursor(0, 0);
           lcdMenu.printMenu("Set home point?");
           confirmZeroPoint = true;
         }
-        break;
-    }
+      }
+      break;
+
+    case btnNONE: {
+        processKeyStateChanges(btnNONE, 0);
+      }
+      break;
   }
 
-  return true;
+  mount.loop();
+
+  return false;
 }