add comment pilatus

debye_bec/devices/pilatus/pilatus.py

@@ -490,6 +490,9 @@ class Pilatus(PSIDeviceBase, ADBase):
             # Compute number of triggers
             total_trig_lo, total_trig_hi = self._calculate_trigger(scan_msg)
             # Set the number of images, we may also set this to a higher values if preferred and stop the acquisition
+            # TODO This logic is prone to errors, as we rely on the scans to nicely resolve to n_images. We should
+            # use here instead a way of settings the n_images independently of the scan parameters to avoid running out of sync
+            # with the complete method. Ideally we comput them in the scan itself.. This is much safer IMO!
             self.n_images = (total_trig_lo + total_trig_hi) * self.scan_parameter.n_of_trigger
             exp_time = self.scan_parameter.exp_time
             self.trigger_source.set(MONOTRIGGERSOURCE.INPOS).wait(5)