gmoccapy: fix "spindle speed override wasn't applied in reverse direction"

Author: hansu

src/emc/usr_intf/gmoccapy/gmoccapy.py

@@ -3737,7 +3737,7 @@ def _update_spindle(self):
         else:
             speed = self.stat.spindle[0]['speed']
         self.widgets.active_speed_label.set_label("{0:.0f}".format(abs(speed)))
-        self.widgets.lbl_spindle_act.set_text("S {0}".format(int(speed * self.spindle_override)))
+        self.widgets.lbl_spindle_act.set_text("S {0}".format(int(round(speed * self.spindle_override))))
 
     def _update_vc(self):
         if self.stat.spindle[0]['direction'] != 0:
@@ -3864,11 +3864,11 @@ def on_spc_spindle_value_changed(self, widget, data=None):
                 speed = abs(self.stat.spindle[0]['speed'])
             spindle_override = value / 100
             real_spindle_speed = speed * spindle_override
-            if real_spindle_speed > self.max_spindle_rev:
-                value_to_set = value / (real_spindle_speed / self.max_spindle_rev)
+            if abs(real_spindle_speed) > self.max_spindle_rev:
+                value_to_set = value / (abs(real_spindle_speed) / self.max_spindle_rev)
                 real_spindle_speed = self.max_spindle_rev
-            elif real_spindle_speed < self.min_spindle_rev:
-                value_to_set = value / (real_spindle_speed / self.min_spindle_rev)
+            elif abs(real_spindle_speed) < self.min_spindle_rev:
+                value_to_set = value / (abs(real_spindle_speed) / self.min_spindle_rev)
                 real_spindle_speed = self.min_spindle_rev
             else:
                 value_to_set = spindle_override * 100

src/emc/usr_intf/gmoccapy/release_notes.txt

@@ -1,3 +1,9 @@
+
+- bug in spindle speed override: The override value wasn't applied
+  when spindle turning in reverse direction
+
+  Thanks Reinhard for reporting
+
 ver 3.1.3.8 (LinuxCNC 2.8.3 release)
 - fixed translation of some tooltips and a dialog message
   Thanks to zz912