[evt] improve function expression parsing and remove unneeded typecast

Author: gipert

src/pygama/evt/build_evt.py

@@ -414,7 +414,12 @@ def evaluate_expression(
         #     pygama.evt.modules.spms.my_func([...], arg1=val, arg2=val)
 
         # get arguments list passed to the function (outermost parentheses)
-        args_str = re.search(r"\((.*)\)$", expr.strip()).group(1)
+        result = re.search(r"\((.*)\)$", expr.strip(), re.DOTALL)
+        if result is None:
+            msg = f"could not parse the function arguments in '{expr}'"
+            raise RuntimeError(msg)
+
+        args_str = result.group(1)
 
         # handle tier scoping: evt.<>
         args_str = args_str.replace("evt.", "")

src/pygama/evt/modules/spms.py

@@ -140,7 +140,7 @@ def gather_pulse_data(
             pulse_mask = pulse_mask.view_as("ak")
 
         # apply the mask
-        data = data[ak.values_astype(pulse_mask, bool)]
+        data = data[pulse_mask]
 
     # remove empty arrays = table_names with no pulses
     if drop_empty: