première version avec boite de dialogue toolChange alpha

Author: fra589

cn5X.py

@@ -105,9 +105,6 @@ def __init__(self, parent=None):
     self.logDebug.document().setMaximumBlockCount(2000) # Limite la taille des logs a 2000 lignes
     self.ui.qtabConsole.setCurrentIndex(CN5X_TAB_LOG)               # Active le tab de la log cn5X++
 
-    self.__gcodeFile = gcodeFile(self.ui, self.ui.gcodeTable)
-    self.__gcodeFile.sig_log.connect(self.on_sig_log)
-
     self.timerDblClic = QtCore.QTimer()
 
     self.__grblCom = grblCom()
@@ -127,23 +124,28 @@ def __init__(self, parent=None):
     self.__grblCom.sig_serialLock.connect(self.on_sig_serialLock)
 
     self.__beeper = cn5XBeeper();
-    
+
     self.__arretUrgence     = True
     def arretUrgence():
       return self.__arretUrgence
 
     self.__decode = grblDecode(self.ui, self.log, self.__grblCom, self.__beeper, arretUrgence)
     self.__grblCom.setDecodeur(self.__decode)
 
+    # Boite de dialogue de changement d'outils
+    self.__dlgToolChange = dlgToolChange(self, self.__grblCom, self.__decode, DEFAULT_NB_AXIS, DEFAULT_AXIS_NAMES)
+    self.__dlgToolChange.setParent(self)
+
+    self.__gcodeFile = gcodeFile(self.ui, self.ui.gcodeTable, self.__dlgToolChange)
+    self.__gcodeFile.sig_log.connect(self.on_sig_log)
+
     self.__jog = grblJog(self.__grblCom)
     self.ui.dsbJogSpeed.setValue(DEFAULT_JOG_SPEED)
     self.ui.dsbJogSpeed.valueChanged.connect(self.on_dsbJogSpeed_valueChanged)
 
     self.__probe = grblProbe(self.__grblCom)
     self.__probe.sig_log.connect(self.on_sig_log)
     self.__probeResult       = None
-    self.__initialToolLenght = False
-    self.__initialProbeZ     = False
 
     self.__connectionStatus = False
     self.__cycleRun         = False
@@ -203,6 +205,11 @@ def arretUrgence():
     # Flag pour unicité de la boite de dialogue Jog
     self.dlgJog = None
 
+    self.iconLinkOn  = QtGui.QIcon()
+    self.iconLinkOff = QtGui.QIcon()
+    self.iconLinkOn.addPixmap(QtGui.QPixmap(":/cn5X/images/btnLinkOn.svg"), QtGui.QIcon.Normal, QtGui.QIcon.On)
+    self.iconLinkOff.addPixmap(QtGui.QPixmap(":/cn5X/images/btnLinkOff.svg"), QtGui.QIcon.Normal, QtGui.QIcon.Off)
+
     '''---------- Connections des evennements de l'interface graphique ----------'''
 
     self.ui.btnUrgence.pressed.connect(self.on_arretUrgence)             # Evenements du bouton d'arret d'urgence
@@ -369,6 +376,7 @@ def arretUrgence():
 
     # Boutons de probe Z
     self.ui.btnProbeZ.clicked.connect(self.on_btnProbeZ)
+    self.ui.btnSaveToolChangePosition.clicked.connect(self.on_btnSaveToolChangePosition)
     self.ui.btnGoToSensor.clicked.connect(self.on_btnGoToSensor)
     self.ui.btnG49.clicked.connect(self.on_btnG49)
     self.ui.btnG43_1.clicked.connect(self.on_btnG43_1)
@@ -871,20 +879,12 @@ def on_dlgJogFinished(self):
 
   def on_mnuToolChange(self):
     ''' Appel de la boite de dialogue de changement d'outils '''
-    self.dlgToolChange = dlgToolChange(self.__grblCom, self.__decode, self.__nbAxis, self.__axisNames)
-    self.dlgToolChange.setParent(self)
-    self.dlgToolChange.sig_close.connect(self.on_dlgToolChangeFinished)
-    RC = self.dlgToolChange.showDialog()
+    RC = self.__dlgToolChange.showDialog()
     if RC == QtWidgets.QDialog.Accepted:
       print("Changement d'outil OK")
     else:
       print("Changement d'outil annulé")
 
-  def on_dlgToolChangeFinished(self):
-    ''' Supression de la boite de dialogue après fermeture '''
-    self.dlgToolChange.sig_close.disconnect(self.on_dlgToolChangeFinished)
-    self.dlgToolChange = None
-
 
   @pyqtSlot()
   def on_mnuPredefinedLocations(self):
@@ -992,8 +992,9 @@ def on_qtabMain_currentChanged(self, tabIndex):
         self.ui.rbtDefineOriginZ_G54.setChecked(self.__settings.value("Probe/DefineOriginZ_G54", DEFAULT_PROBE_ORIGINE_G54_Z, type=bool))
         self.ui.rbtDefineOriginZ_G92.setChecked(self.__settings.value("Probe/DefineOriginZ_G92", DEFAULT_PROBE_ORIGINE_G92_Z, type=bool))
         self.ui.dsbOriginOffsetZ.setValue(self.__settings.value("Probe/OriginOffsetZ", DEFAULT_PROBE_ORIGINE_OFFSET_Z, type=float))
-        self.ui.dsbToolLengthSensorX.setValue(self.__settings.value("Probe/ToolSensorPositionX", DEFAULT_TOOLSENSOR_POSITION_X, type=float))
-        self.ui.dsbToolLengthSensorY.setValue(self.__settings.value("Probe/ToolSensorPositionY", DEFAULT_TOOLSENSOR_POSITION_Y, type=float))
+        self.ui.dsbToolLengthSensorZ.setValue(self.__settings.value("Probe/ToolChangePositionZ", DEFAULT_TOOLCHANGE_POSITION_Z, type=float))
+        self.ui.dsbToolLengthSensorX.setValue(self.__settings.value("Probe/ToolChangePositionX", DEFAULT_TOOLCHANGE_POSITION_X, type=float))
+        self.ui.dsbToolLengthSensorY.setValue(self.__settings.value("Probe/ToolChangePositionY", DEFAULT_TOOLCHANGE_POSITION_Y, type=float))
 
 
   @pyqtSlot()
@@ -1024,6 +1025,7 @@ def on_btnProbeZ(self):
         self.__probeResult = self.__probe.g38(P=3, F=probeSeekRate, Z=-probeDistance, g2p=False)
         # On mémorise le résultat
         self.ui.lblLastProbZ.setText('{:+0.3f}'.format(float(self.__probeResult.getAxisByName("Z"))))
+        self.__dlgToolChange.di.lblLastProbZ.setText('{:+0.3f}'.format(float(self.__probeResult.getAxisByName("Z"))))
         # On retract d'une distance probePullOff
         retractGCode = "G0Z{:+0.3f}".format(probePullOff)
         self.__grblCom.gcodePush(retractGCode)
@@ -1050,6 +1052,7 @@ def on_btnProbeZ(self):
       self.__probeResult = self.__probe.g38(P=3, F=probeFeedRate, Z=-fineProbeDistance, g2p=go2point)
       # On mémorise le résultat précis
       self.ui.lblLastProbZ.setText('{:+0.3f}'.format(float(self.__probeResult.getAxisByName("Z"))))
+      self.__dlgToolChange.di.lblLastProbZ.setText('{:+0.3f}'.format(float(self.__probeResult.getAxisByName("Z"))))
 
       if self.ui.rbtRetractAfterZ.isChecked():
         # On retract d'une distance probeRetract
@@ -1082,9 +1085,9 @@ def on_btnProbeZ(self):
       self.__grblCom.gcodePush(oldG90_91)
 
     if (self.__probeResult is not None) and (self.__probeResult.isProbeOK()):
-      self.__initialProbeZ = True
-      if self.__initialToolLenght:
-        self.calculateToolOffset()
+      self.__dlgToolChange.setInitialProbeZ(True)
+      if self.__dlgToolChange.initialToolLenght():
+        self.__dlgToolChange.calculateToolOffset()
 
     # Pour finir, on sauvegarde les derniers paramètres de probe dans les settings
     self.__settings.setValue("Probe/DistanceZ", self.ui.dsbDistanceZ.value())
@@ -1098,17 +1101,41 @@ def on_btnProbeZ(self):
     self.__settings.setValue("Probe/RetractDistanceZ", self.ui.dsbRetractZ.value())
 
 
+  @pyqtSlot()
+  def on_btnSaveToolChangePosition(self):
+    '''
+    Déplacement rapide du palpeur de longueur d'outil vers les coordonnées machine :
+    Z d'abord pour dégager, puis X, Y.
+    '''
+    # Recupération des coordonnées Z, X & Y du point
+    posZ = self.ui.dsbToolLengthSensorZ.value()
+    posX = self.ui.dsbToolLengthSensorX.value()
+    posY = self.ui.dsbToolLengthSensorY.value()
+    # Memorise la position dans les settings
+    self.__settings.setValue("Probe/ToolChangePositionZ", posZ)
+    self.__settings.setValue("Probe/ToolChangePositionX", posX)
+    self.__settings.setValue("Probe/ToolChangePositionY", posY)
+
+
   @pyqtSlot()
   def on_btnGoToSensor(self):
-    ''' Déplacement vers les coordonnées machine X, Y du palpeur de longueur d'outil'''
-    # Recupération des coordonnées X & Y du point
+    '''
+    Déplacement rapide du palpeur de longueur d'outil vers les coordonnées machine :
+    Z d'abord pour dégager, puis X, Y.
+    '''
+    # Recupération des coordonnées Z, X & Y du point
+    posZ = self.ui.dsbToolLengthSensorZ.value()
     posX = self.ui.dsbToolLengthSensorX.value()
     posY = self.ui.dsbToolLengthSensorY.value()
-    deplacementGCode = "G53G0X{}Y{}".format(posX, posY)
-    self.__grblCom.gcodePush(deplacementGCode)
+    # Effectue les déplacements
+    deplacementGCodeZ  = "G53G0Z{}".format(posZ)
+    deplacementGCodeXY = "G53G0X{}Y{}".format(posX, posY)
+    self.__grblCom.gcodePush(deplacementGCodeZ)
+    self.__grblCom.gcodePush(deplacementGCodeXY)
     # Memorise la position dans les settings
-    self.__settings.setValue("Probe/ToolSensorPositionX", posX)
-    self.__settings.setValue("Probe/ToolSensorPositionY", posY)
+    self.__settings.setValue("Probe/ToolChangePositionZ", posZ)
+    self.__settings.setValue("Probe/ToolChangePositionX", posX)
+    self.__settings.setValue("Probe/ToolChangePositionY", posY)
 
 
   @pyqtSlot()
@@ -1117,7 +1144,7 @@ def on_btnG49(self):
     Mémorise le Z du point de contact initial de l'outil pour calculer les outils suivants
     et envoi G49 pour réinitialiser une éventuelle longueur précédente.
     '''
-    if not self.__initialProbeZ:
+    '''if not self.__dlgToolChange.__initialProbeZ:
       self.log(logSeverity.error.value, self.tr("on_btnG49(): No initial Z probe result, can't get initial tool length probe!"))
       m = msgBox(
                   title  = self.tr("Error !"),
@@ -1132,18 +1159,20 @@ def on_btnG49(self):
     
     # Initialise la longueur d'outil initiale
     self.ui.lblInitToolLength.setText(self.ui.lblLastProbZ.text())
+    self.__dlgToolChange.di.lblInitToolLength.setText(self.__dlgToolChange.di.lblLastProbZ.text())
     self.__grblCom.gcodePush("G49")
 
-    self.__initialToolLenght = True
-
+    self.__dlgToolChange.__initialToolLenght = True
+    '''
+    self.__dlgToolChange.on_btnG49()
 
   @pyqtSlot()
   def on_btnG43_1(self):
     '''
     Calcul de la correction de longueur d'outil par rapport à la valeur initiale mémorisée
     et configure le "Tool Length Offset" dans Grbl à l'aide de G43.1
     '''
-    if not self.__initialToolLenght:
+    '''if not self.__dlgToolChange.__initialToolLenght:
       self.log(logSeverity.error.value, self.tr("on_btnG43_1(): No initial tool length, can't calculate length offset!"))
       m = msgBox(
                   title  = self.tr("Error !"),
@@ -1156,9 +1185,11 @@ def on_btnG43_1(self):
       m.afficheMsg()
       return
     # Envoi de la correction de longueur d'outil
-    toolOffset = self.calculateToolOffset()
+    toolOffset = self.__dlgToolChange.calculateToolOffset()
     toolOffsetGcode = "G43.1Z{}".format(toolOffset)
     self.__grblCom.gcodePush(toolOffsetGcode)
+    '''
+    self.__dlgToolChange.on_btnG43_1()
 
 
   @pyqtSlot()
@@ -1810,16 +1841,6 @@ def on_chkSeekXY(self):
       self.ui.dsbPullOffXY.setEnabled(False)
 
 
-  @pyqtSlot()
-  def calculateToolOffset(self):
-    # Traitement de la correction de longueur d'outil.
-    lastProbe         = float(self.ui.lblLastProbZ.text().replace(' ', ''))
-    initialToolLength = float(self.ui.lblInitToolLength.text().replace(' ', ''))
-    toolOffset = lastProbe - initialToolLength
-    self.ui.lblToolOffset.setText('{:+0.3f}'.format(toolOffset))
-    return toolOffset
-
-
   @pyqtSlot(str)
   def on_sig_config_changed(self, data: str):
     self.log(logSeverity.info.value, self.tr("Grbl configuration updated: {}").format(data))
@@ -1925,10 +1946,15 @@ def on_spindleOverride(self, value: int):
 
   @pyqtSlot()
   def on_btnLinkOverride(self):
-    if self.ui.btnLinkOverride.isChecked() and (self.ui.dialAvance.value() != self.ui.dialBroche.value()):
-      newValue = (self.ui.dialAvance.value() + self.ui.dialBroche.value()) / 2
-      self.ui.dialBroche.setValue(newValue)
-      self.ui.dialAvance.setValue(newValue)
+    if (self.ui.btnLinkOverride.isChecked()):
+      self.ui.btnLinkOverride.setIcon(self.iconLinkOn)
+      if (self.ui.dialAvance.value() != self.ui.dialBroche.value()):
+        # On force comme valeur la moyenne des 2
+        newValue = int((self.ui.dialAvance.value() + self.ui.dialBroche.value()) / 2)
+        self.ui.dialBroche.setValue(newValue)
+        self.ui.dialAvance.setValue(newValue)
+    else: # Bouton non checked
+      self.ui.btnLinkOverride.setIcon(self.iconLinkOff)
 
 
   @pyqtSlot()
@@ -2146,9 +2172,12 @@ def on_sig_log(self, severity: int, data: str):
       self.logCn5X.append(time.strftime("%Y-%m-%d %H:%M:%S") + " : Error   : " + data)
       if not self.ui.btnDebug.isChecked():
         self.ui.qtabConsole.setCurrentIndex(CN5X_TAB_LOG)
+
+
   def log(self, severity: int, data: str):
     self.on_sig_log(severity, data)
 
+
   @pyqtSlot(str)
   def on_sig_init(self, data: str):
     self.log(logSeverity.info.value, self.tr("cn5X++ : Grbl initialized."))
@@ -2219,14 +2248,15 @@ def on_sig_config(self, data: str):
     if data[:5] == "[AXS:":
       self.__nbAxis           = int(data[1:-1].split(':')[1])
       self.__axisNames        = list(data[1:-1].split(':')[2])
+      # Mise à jour classe grblProbe
+      self.__probe.setAxisNames(self.__axisNames)
+      # Mise à jour classe dlgToolChange
+      self.__dlgToolChange.setAxisNumber(self.__nbAxis)
+      self.__dlgToolChange.setAxisNames(self.__axisNames)
       if len(self.__axisNames) < self.__nbAxis:
         # Il est posible qu'il y ait moins de lettres que le nombre d'axes si Grbl
         # implémente l'option REPORT_VALUE_FOR_AXIS_NAME_ONCE
         self.__nbAxis = len(self.__axisNames);
-      '''self.updateAxisNumber()
-      self.__decode.setNbAxis(self.__nbAxis)'''
-      # Mise à jour classe grblProbe
-      self.__probe.setAxisNames(self.__axisNames)
 
     # Memorise les courses maxi pour calcul des jogs max.
     elif data[:4] == "$130":

cn5X.qrc

@@ -66,9 +66,9 @@
     <file>images/ledRougeAlumee.svg</file>
     <file>images/ledRougeEteinte.svg</file>
     <file>images/btnKillAlarm.svg</file>
-    <file>images/btnLink.svg</file>
-    <file>images/btnLinkOff.svg</file>
     <file>images/btnResetSpeed.svg</file>
+    <file>images/btnLinkOff.svg</file>
+    <file>images/btnLinkOn.svg</file>
     <file>images/btnPause.svg</file>
     <file>images/btnPause_down.svg</file>
     <file>images/btnPause_light.svg</file>

cn5X_config.py

@@ -26,7 +26,7 @@
 
 ORG_NAME           = "fra589"
 APP_NAME           = "cn5X++"
-APP_VERSION_STRING = "0.8.9"
+APP_VERSION_STRING = "0.8.10"
 APP_VERSION_DATE   = "20220509"
 
 DEFAULT_NB_AXIS    = 3 # Laisser 3 permet de gerer un Grbl original a 3 axes
@@ -148,8 +148,9 @@ class logSeverity(Enum):
 DEFAULT_PROBE_ORIGINE_G54_Z             = True
 DEFAULT_PROBE_ORIGINE_G92_Z             = False
 DEFAULT_PROBE_ORIGINE_OFFSET_Z          = 0.0
-DEFAULT_TOOLSENSOR_POSITION_X           = -5
-DEFAULT_TOOLSENSOR_POSITION_Y           = -5
+DEFAULT_TOOLCHANGE_POSITION_Z           = -5
+DEFAULT_TOOLCHANGE_POSITION_X           = -5
+DEFAULT_TOOLCHANGE_POSITION_Y           = -5
 
 ''' Valeurs renvoyées par grblDecode.waitForGrblReply() et grblDecode.waitForGrblProbe() '''
 SIG_OK    = 0

cn5X_gcodeFile.py

@@ -49,13 +49,14 @@ class gcodeFile(QObject):
 
   sig_log     = pyqtSignal(int, str) # Message de fonctionnement du composant
 
-  def __init__(self, ui, gcodeFileUi: QListView):
+  def __init__(self, ui, gcodeFileUi: QListView, dialogToolChange: dlgToolChange):
     super().__init__()
     self.__filePath         = ""
     self.__ui               = ui
     self.__gcodeFileUi      = gcodeFileUi
     self.__gcodeFileUiModel = QStandardItemModel(self.__gcodeFileUi)
     self.__gcodeFileUiModel.itemChanged.connect(self.on_gcodeChanged)
+    self.__dlgToolChange = dialogToolChange
 
     self.__gcodeCharge      = False
     self.__gcodeChanged     = False
@@ -268,13 +269,9 @@ def setGcodeChanged(self, value:bool):
     self.__gcodeChanged = value
 
 
-  def showToolChange(self):
+  def toolChange(self):
     ''' Appel de la boite de dialogue de changement d'outils '''
-    self.dlgToolChange = dlgToolChange(self.__grblCom, self.__decode, self.__nbAxis, self.__axisNames)
-    self.dlgToolChange.setParent(self)
-    self.dlgToolChange.sig_close.connect(self.on_dlgToolChangeFinished)
-    
-    RC = self.dlgToolChange.showDialog()
+    RC = self.__dlgToolChange.showDialog()
     if RC == QtWidgets.QDialog.Accepted:
       print("Changement d'outil OK")
       return True
@@ -283,9 +280,5 @@ def showToolChange(self):
       return False
 
 
-  def on_dlgToolChangeFinished(self):
-    ''' Supression de la boite de dialogue après fermeture '''
-    self.dlgToolChange.sig_close.disconnect(self.on_dlgToolChangeFinished)
-    self.dlgToolChange = None