Merge pull request #440 from smoia/fix/trigger_idx_auto

Fix channel trigger variable type when the trigger is automatically detected

Author: smoia

phys2bids/physio_obj.py

@@ -590,6 +590,7 @@ def auto_trigger_selection(self):
                 Automatically retrieved trigger index
         """
         LGR.info('Running automatic trigger detection.')
+        LGR.info('Matching channel names with known trigger names first.')
         joint_match = '§'.join(TRIGGER_NAMES)
         indexes = []
         for n, case in enumerate(self.ch_name):
@@ -604,10 +605,10 @@ def auto_trigger_selection(self):
                 raise Exception('More than one possible trigger channel was automatically found. '
                                 'Please run phys2bids specifying the -chtrig argument.')
             else:
-                self.trigger_idx = indexes[0]
+                self.trigger_idx = int(indexes[0])
         else:
             # Time-domain automatic trigger detection
-
+            LGR.info('Find the trigger channel by measuring data distance from its value limits.')
             # Create numpy array with all channels (excluding time)
             channel_ts = np.array(self.timeseries[1:])
 
@@ -621,7 +622,7 @@ def auto_trigger_selection(self):
             distance_mean = np.mean(distance, axis=1)
 
             # Set the trigger as the channel with the smallest distance
-            self.trigger_idx = np.nanargmin(distance_mean) + 1
+            self.trigger_idx = int(np.nanargmin(distance_mean) + 1)
 
         LGR.info(f'{self.ch_name[self.trigger_idx]} selected as trigger channel')