V1.6.17 - Updates

- Made calibration use two bytes for storage, so you can set adjust more than a factor 1.0255
- Renamed the Polaris RA coordinates.
- Forced users to choose the correct RA ring before compiling.

Author: ClutchplateDude

Software/Arduino code/OpenAstroTracker/OpenAstroTracker.ino

@@ -1,7 +1,7 @@
 /*
   =======================================================================================================================================
 
-    Version 1.6.15
+    Version 1.6.17
 
     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)
@@ -14,7 +14,7 @@
 
   =======================================================================================================================================
 */
-String version = "V1.6.16";
+String version = "V1.6.17";
 
 // See NORTHERN_HEMISPHERE in Globals.h if you not in the northern hemisphere
 
@@ -24,9 +24,9 @@ String version = "V1.6.16";
 // One RA revolution needs 26.43 (1057.1mm / 40mm) stepper revolutions (V2: 28.27 (1131mm/40mm))
 // Which means 108245 steps (26.43 x 4096) moves 360 degrees (V2: 115812 steps (28.27 x 4096))
 // So there are 300.1 steps/degree (108245 / 360)  (V2: 322 (115812 / 360))
-// Theoretical RA tracking speed is 1.246586 (300 x 14.95903 / 3600) (V2 : 1.333800 (322 x 14.95903 / 3600) steps/sec
-
-int RAStepsPerDegree = 300;      // V1 Ring has a ridge that the belt runs on and the ring runs on the bearings
+// Theoretically correct RA tracking speed is 1.246586 (300 x 14.95903 / 3600) (V2 : 1.333800 (322 x 14.95903 / 3600) steps/sec
+#error "Please uncomment one of the two following lines depending on which version of the RA ring you printed. And comment out this line."
+// int RAStepsPerDegree = 300;      // V1 Ring has a ridge on top of the ring that the belt runs on and the ring runs on the bearings
 // int RAStepsPerDegree = 322;      // V2 Ring has belt in a groove and belt runs on bearings
 
 
@@ -61,9 +61,9 @@ float DECStepperDownLimit = 10000;    // Going much more than this will make the
 float DECStepperUpLimit = -22000;     // Going much more than this is going below the horizon.
 
 // These values are needed to calculate the current position during initial alignment.
-int h = 21;
-int m = 2;
-int s = 25;
+int PolarisRAHour = 21;
+int PolarisRAMinute = 2;
+int PolarisRASecond = 25;
 // You get these values by calculating 24h minus the current (not J2000) RA of Polaris.
 // This changes slightly over weeks, so adjust every couple of months.
 // This value is from 27.Nov.19, I suggest next adjustment in mid 2020

Software/Arduino code/OpenAstroTracker/b_setup.ino

@@ -35,14 +35,14 @@ void setup() {
 
   // Configure the mount
   // Set the global HA correction
-  mount.setHACorrection(h, m, s);
+  mount.setHACorrection(PolarisRAHour, PolarisRAMinute, PolarisRASecond);
 
   // Set the stepper motor parameters
   mount.configureRAStepper(FULLSTEP, RAmotorPin1, RAmotorPin2, RAmotorPin3, RAmotorPin4, RAspeed, RAacceleration);
   mount.configureDECStepper(HALFSTEP, DECmotorPin1, DECmotorPin2, DECmotorPin3, DECmotorPin4, DECspeed, DECacceleration);
 
-  // Read persisted values adn set in mount
-  inputcal = EEPROM.read(0);
+  // Read persisted values and set in mount
+  inputcal = EEPROM.read(0) + EEPROM.read(3) * 256;
   DayTime haTime = DayTime(EEPROM.read(1), EEPROM.read(2), 0);
   mount.setSpeedCalibration(speed + inputcal / 10000);
 #ifdef DEBUG_MODE
@@ -76,9 +76,9 @@ void setup() {
   }
 
   lcdMenu.updateDisplay();
-  
+
 #ifdef DEBUG_MODE
   Serial.println("SetupDone");
 #endif
-  
+
 }

Software/Arduino code/OpenAstroTracker/c76_menuCAL.ino

@@ -15,15 +15,19 @@ bool processCalibrationKeys() {
   if (calState == SPEED_CALIBRATION)
   {
     if (lcdButtons.currentState() == btnUP)  {
-      inputcal += 1;  //0.0001;
-      mount.setSpeedCalibration(speed + inputcal / 10000);
+      if (inputcal < 32760) { // Don't overflow 16 bit signed
+        inputcal += 1;  //0.0001;
+        mount.setSpeedCalibration(speed + inputcal / 10000);
+      }
       mount.delay(calDelay);
       calDelay = max(5, 0.96 * calDelay);
       checkForKeyChange = false;
     }
     else if (lcdButtons.currentState() == btnDOWN)  {
-      inputcal -= 1 ; //0.0001;
-      mount.setSpeedCalibration(speed + inputcal / 10000);
+      if (inputcal > -32760) { // Don't overflow 16 bit signed
+        inputcal -= 1 ; //0.0001;
+        mount.setSpeedCalibration(speed + inputcal / 10000);
+      }
       mount.delay(calDelay);
       calDelay = max(5, 0.96 * calDelay);
       checkForKeyChange = false;
@@ -64,7 +68,9 @@ bool processCalibrationKeys() {
       case SPEED_CALIBRATION: {
           // UP and DOWN are handled above
           if (key == btnSELECT) {
-            EEPROM.update(0, inputcal);
+            int cal = floor(inputcal);
+            EEPROM.update(0, cal & 0x00FF);
+            EEPROM.update(3, (cal & 0xFF00) >> 8);
             mount.setSpeedCalibration(speed + inputcal / 10000);
             lcdMenu.printMenu("Stored.");
             mount.delay(500);