V1.8.06 - Updates

- DEC axis was inadvertently inverted by default after merge. Fixed.
- Guide pulses were not 2x sidereal and 0 for default steppers. Fixed.

Author: ClutchplateDude

Software/Arduino code/OpenAstroTracker/Configuration.hpp

@@ -18,7 +18,7 @@
   =======================================================================================================================================
 */
 
-String version = "V1.8.05";
+String version = "V1.8.06";
 
 ///////////////////////////////////////////////////////////////////////////
 // Also use Configuration_adv for further adjustments!

Software/Arduino code/OpenAstroTracker/Configuration_adv.hpp

@@ -63,7 +63,7 @@
 // INVERT AXIS
 // Set to 1 or 0 to invert motor directions
 #define INVERT_RA_DIR 0 
-#define INVERT_DEC_DIR 1
+#define INVERT_DEC_DIR 0
 //
 //
 ////////////////////////////
@@ -78,8 +78,13 @@
 // GUIDE SETTINGS
 // This is the multiplier of the normal trackingspeed that a pulse will have 
 // standard value: RA 2.2;  DEC 1.2
+#if RA_STEPPER_TYPE == STEP_28BYJ48
+#define RA_PULSE_MULTIPLIER 2.0
+#define DEC_PULSE_MULTIPLIER 1.0
+#else
 #define RA_PULSE_MULTIPLIER 1.5
-//#define DEC_PULSE_MULTIPLIER 1.2
+#define DEC_PULSE_MULTIPLIER 1.2
+#endif
 
 
 

Software/Arduino code/OpenAstroTracker/Mount.cpp

@@ -854,15 +854,15 @@ void Mount::guidePulse(byte direction, int duration) {
   switch (direction) {
     case NORTH:
     _stepperDEC->setAcceleration(2500);
-    _stepperDEC->setMaxSpeed(decTrackingSpeed * 1.2);
-    _stepperDEC->setSpeed(decTrackingSpeed);
+    _stepperDEC->setMaxSpeed(decTrackingSpeed * (DEC_PULSE_MULTIPLIER + 0.2));
+    _stepperDEC->setSpeed(decTrackingSpeed * DEC_PULSE_MULTIPLIER);
     _mountStatus |= STATUS_GUIDE_PULSE | STATUS_GUIDE_PULSE_DEC;
     break;
 
     case SOUTH:
     _stepperDEC->setAcceleration(2500);
-    _stepperDEC->setMaxSpeed(decTrackingSpeed * 1.2);
-    _stepperDEC->setSpeed(-decTrackingSpeed);
+    _stepperDEC->setMaxSpeed(decTrackingSpeed * (DEC_PULSE_MULTIPLIER + 0.2));
+    _stepperDEC->setSpeed(-decTrackingSpeed * DEC_PULSE_MULTIPLIER);
     _mountStatus |= STATUS_GUIDE_PULSE | STATUS_GUIDE_PULSE_DEC;
     break;
 
@@ -874,8 +874,12 @@ void Mount::guidePulse(byte direction, int duration) {
 
     case EAST:
     _stepperTRK->setMaxSpeed(raTrackingSpeed * (RA_PULSE_MULTIPLIER + 0.2));
-    //_stepperTRK->setSpeed(raTrackingSpeed + (-RA_PULSE_MULTIPLIER * (raTrackingSpeed / 2.0)));
-    _stepperTRK->setSpeed(0.6 * raTrackingSpeed);
+    #if RA_STEPPER_TYPE == STEP_28BYJ48
+      _stepperTRK->setSpeed(0);
+    #else
+      // Not sure why we don't stop tracking with NEMAs as is customary.....
+      _stepperTRK->setSpeed(0.6 * raTrackingSpeed);
+    #endif
     _mountStatus |= STATUS_GUIDE_PULSE | STATUS_GUIDE_PULSE_RA;
     break;
   }