From 8817de457e49c8ae342373a248afa629729b51eb Mon Sep 17 00:00:00 2001
From: chriscyyeung <chriscyyeung@gmail.com>
Date: Tue, 14 Sep 2021 17:13:34 -0400
Subject: [PATCH] Re #1: Add auto-update function

---
 MarkupToModel/MarkupToModel.py                |  20 +++++++++++++++-
 MarkupToModel/Resources/UI/MarkupToModel.ui   |  22 ++++++++++++++----
 .../__pycache__/MarkupToModel.cpython-36.pyc  | Bin 12092 -> 12713 bytes
 3 files changed, 37 insertions(+), 5 deletions(-)

diff --git a/MarkupToModel/MarkupToModel.py b/MarkupToModel/MarkupToModel.py
index 74b8e61..1ed8150 100644
--- a/MarkupToModel/MarkupToModel.py
+++ b/MarkupToModel/MarkupToModel.py
@@ -135,6 +135,10 @@ def setup(self):
     self.ui.outputSelector.connect("currentNodeChanged(vtkMRMLNode*)", self.updateParameterNodeFromGUI)
     self.ui.opacitySlider.connect("valueChanged(double)", self.updateParameterNodeFromGUI)
 
+    self.observedMarkupNode = None
+    self.markupsObserverTag = None
+    self.ui.autoUpdateCheckBox.connect("toggled(bool)", self.onEnableAutoUpdate)
+
     # Buttons
     self.ui.applyButton.connect('clicked(bool)', self.onApplyButton)
 
@@ -236,10 +240,24 @@ def updateParameterNodeFromGUI(self, caller=None, event=None):
 
     self._parameterNode.SetNodeReferenceID("InputVolume", self.ui.inputSelector.currentNodeID)
     self._parameterNode.SetNodeReferenceID("OutputVolume", self.ui.outputSelector.currentNodeID)
-    self.ui.outputSelector.currentNode().GetDisplayNode().SetOpacity(self.ui.opacitySlider.value)  #TODO: bugged
+    self.ui.outputSelector.currentNode().GetDisplayNode().SetOpacity(self.ui.opacitySlider.value)
 
     self._parameterNode.EndModify(wasModified)
 
+  def onEnableAutoUpdate(self, autoUpdate):
+    if self.markupsObserverTag:
+      self.observedMarkupNode.RemoveObserver(self.markupsObserverTag)
+      self.observedMarkupNode = None
+      self.markupsObserverTag = None
+    if autoUpdate and self.ui.inputSelector.currentNode:
+      self.observedMarkupNode = self.ui.inputSelector.currentNode()
+      self.markupsObserverTag = self.observedMarkupNode.AddObserver(
+        slicer.vtkMRMLMarkupsNode.PointModifiedEvent, self.onMarkupsUpdated
+      )
+  
+  def onMarkupsUpdated(self, caller=None, event=None):
+    self.onApplyButton()
+
   def onApplyButton(self):
     """
     Run processing when user clicks "Apply" button.
diff --git a/MarkupToModel/Resources/UI/MarkupToModel.ui b/MarkupToModel/Resources/UI/MarkupToModel.ui
index 91a3c6b..6488039 100644
--- a/MarkupToModel/Resources/UI/MarkupToModel.ui
+++ b/MarkupToModel/Resources/UI/MarkupToModel.ui
@@ -6,8 +6,8 @@
    <rect>
     <x>0</x>
     <y>0</y>
-    <width>270</width>
-    <height>159</height>
+    <width>274</width>
+    <height>191</height>
    </rect>
   </property>
   <layout class="QVBoxLayout" name="verticalLayout">
@@ -67,14 +67,14 @@
         </property>
        </widget>
       </item>
-      <item row="2" column="0">
+      <item row="3" column="0">
        <widget class="QLabel" name="label_3">
         <property name="text">
          <string>Opacity:</string>
         </property>
        </widget>
       </item>
-      <item row="2" column="1">
+      <item row="3" column="1">
        <widget class="qMRMLSliderWidget" name="opacitySlider">
         <property name="singleStep">
          <double>0.010000000000000</double>
@@ -90,6 +90,20 @@
         </property>
        </widget>
       </item>
+      <item row="4" column="0">
+       <widget class="QLabel" name="label_4">
+        <property name="text">
+         <string>Enable auto-udpate</string>
+        </property>
+       </widget>
+      </item>
+      <item row="4" column="1">
+       <widget class="QCheckBox" name="autoUpdateCheckBox">
+        <property name="text">
+         <string/>
+        </property>
+       </widget>
+      </item>
      </layout>
     </widget>
    </item>
diff --git a/MarkupToModel/__pycache__/MarkupToModel.cpython-36.pyc b/MarkupToModel/__pycache__/MarkupToModel.cpython-36.pyc
index d740ba3fd2501645c39dcedde394ef0edd163ae4..bda12e3bb706f1b3215a2c24ccad2819cda132f6 100644
GIT binary patch
delta 1470
zcmah|O>7%Q6yCS?+M9J8JL{j=&Y#`<5UV5+Ed?YLsFS8@OYQtLO_6C59M3kf<9MU-
zCTf#N6-o{u5IIaGBsjHdD4g2Lih4nUQ!fZ4E*usKkdOcg2@scFIPl&kO$9x$tNG@e
z@4b05``+8nW<M=z$Nhe7B$z4OJ9%H*YtY}8B*~ESz@)s)An#(G;+uxECz33}qT7)7
zvlMG#F=S0F&Jx>FKERH!R@R0}khQZ8RGOK5K}vVoA(7HU^>NKF!>?kt@$O0H-jwEI
z4#qd7azi;$4l@N=>y`|XqqL#oLDqhtbZki=F*nBZuFr@!J0s$}?~HiQpH1b~{g%0~
zP%&8FylGa_>;7V;R9t$zOn26Q^Y_D;ZqC;XzGkp&fiJDD<V<GR`m)1oWBVpPRamg~
z!m4FnTww*v7+ExmOT*?>TQ{qt)xvzm$Q;^4OJJ@ui8+$onp=lPXi^&8suG1LuJDyT
z;1N54ui%E53BK!o4IRZ$u*ItwRm<QX*0p90;HHS^U%)5gTm6ns9%&a3;lwWyye@V^
zEpS)d3H<@n;)AeI$mKzOFtP!Waq^fTm?XGBFeN^Z9#^K3KCC~A_5$QZM=U-xL$;R)
z&LY@e!}${S@3Y7pK7<IdDTMEY6`KYwzKmTG$KpSF@?>BlmS~4L@j{~HIk@R=qC5OE
z+G<S*k|zjmS(C#M7C$7Ozb=!856vzdBRFbwO)Rg(lZa53iEjax7}Ys!Iz(v`_$2DC
zvb(H2VOx%ev&&<7%WAm+xweH7=5RMQ2;|Z}ej4NRQ3U&X)1<L%t{IPuyq}8<*%!&~
z%P@9m3y`9(Sxec;?D_ppsqMGQxLK-NS(BBnlngeyhGn#aX7#{jzh78dcIszqPkM-l
z`{^Vnw@zBm;PMfUS`a}34>aK*q<Os;f0c0C)9qHI#L?pt?+4?r9(*S7b1409B{@3b
ze<jtNO2B$I9rD!W>w2pTU{OTdCK6Y1$A9~Qj#5Cp)i$iixm;Qofq}@dmmHrX$PtVY
zTqJmf;3NTcEvIrj9mR)<Q4%;^i=PwC?WrRjNL3d$jiw$|J*wAvRW+bSG@sV2`ZQO4
zynRoWlOSe;G0~a)Tbai#z4c`3cYp~I==vH4#rIvmz_xhUeKhqE8twQ9H}Ee$Wn3w&
zR;*K{+DfHxjfz?0C875WKv`sZPAC@oX2nKN)<+>ZZRHl;DyF=QG+O_&=P!Vw`01H<
z;I>%qdl44vyM5sHZKJjH08`4}5ci+m0ZU~1O=TC=s@Ur{eYYvl#{|0+?bt1(m#GRW
z5>;`_s-k*0sp$~!#>rM!t0l`aY8KzYC7%%21}^(Hh|*2$gt!S&aj(}c^II6PMT7=>
JT|>ds{{T;vUHSk3

delta 877
zcmZva-%Aux6vyYx?u_H?uDd(?L-&WPsJTDfTuq`NQ4vzg{4Gjbnm=ZPU4N|JU3=K1
zBJCk4N)Ci*5<wtV7&Sx>^-$4EJp?@!(o+xBlR@+cbguLm!+h@dp8Gv_&bfECr(TVz
zO*WhQ-rhE$wLDdy7i$*;K}ZQn(1oO^L(-%N@Kf=L>Y(n?y(^Hk>E(K<UPh`!_v!u>
zA!*meK_MF2b>W+!%a~McB7DGk>yyeQAyb?w%eZwjDc`aP0*(G<0R&y53>imlkF|nr
zY9>83ozf$d+3ZZTCmJwfB?kew>|>RFDn&4>W(}-RB3aFf<OMIy=rL)gf{#Av##YBO
z+imjY#C>-mupiZ&)6j{RoRx6LxaU*>Hu0hM8Xn-ZYfI}SXVk=8G_XOAA#_?@QX9!f
z*zEoV0~q(9#BGN0&T|nwr?{(|qlcrHqYsBlo1_7fZ;czJwa_|Da@X3sn4e$F4`*i<
zXH!v$CAjr8$5D>s+*6dAo0yzQ>Hn#rkJdk9v2Pfz;<#@_WDK``dotgejpL@j9M0fN
ze_H-mb}Ud6_(p3v2SJb>U=~%;4Q_BiP%PB%A4S3^W3}CF%$Az~bX*Aa!yNBd?x2)E
z0)7p(g98H*e^Vm`*h!8qj((07jy4W{9E`Wic#*7~qXXL_WzjgvB)vI)VTy@rP?fME
zD@FT{qSzIWYEd;7$AyT`_)@Vhg5Ahfc|aPY1dnmEdJmu*mkxY{5O&r6fC4s!>tGen
zg)1P1m&5gN9(Tfrr3~4d@n^W(!V}plM}|rWf@F+Fegj=xU+g-p<8JH-j2K!Sm@Ne|
zrC-p`!A3BYxCI&9No1uBijHDcL)Nm+H@VBP!J9d>MzRa28HI!*nMFlXWX7XW7ptLR
uS6Q4(=kuvto?W9U>&EX5XDtg{9^=b?yboS^h_A2Fj-_~|vDOrKbo>R%gvUGp