change logic in refine.py to be less repetitive, uses a ind_list for all

locklost/plugins/refine.py

@@ -94,20 +94,15 @@ def refine_calculate(event, params, window, refined_gps=None, threshold=None):
     else:
         inds_above = np.where(buf.data > upper_threshold)[0]
         inds_below = np.where(buf.data < lower_threshold)[0]
-
-        if inds_above.any() or inds_below.any():
-            if inds_above.any() and inds_below.any():
-                ind_list = [[np.min(inds_above), upper_threshold],
-                            [np.min(inds_below), lower_threshold]]
-                ind_list.sort()
-                ind = ind_list[0][0]
-                threshold = ind_list[0][1]
-            elif inds_above.any() and not inds_below.any():
-                ind = np.min(inds_above)
-                threshold = upper_threshold
-            elif not inds_above.any() and inds_below.any():
-                ind = np.min(inds_below)
-                threshold = lower_threshold
+        ind_list = []
+        if inds_above.any():
+            ind_list.append([np.min(inds_above), upper_threshold])
+        if inds_below.any():
+            ind_list.append([np.min(inds_below), lower_threshold])
+        if ind_list.any():
+            ind_list.sort()
+            ind = ind_list[0][0]
+            threshold = ind_list[0][1]
             refined_gps = buf.tarray[ind]
             logging.info("threshold: {}".format(threshold))
             logging.info("refined time: {}".format(refined_gps))