From be2832cbadce040d515387ad964cf7d49a8c1237 Mon Sep 17 00:00:00 2001
From: Patrick Godwin <patrick.godwin@ligo.org>
Date: Fri, 11 May 2018 11:54:22 -0700
Subject: [PATCH] gstlal_peakfinder.c: fixed issue in
 gstlal_peak_max_over_channels where peak_max was not stored correctly,
 producing nonsensical results

---
 gstlal/lib/gstlal/gstlal_peakfinder.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/gstlal/lib/gstlal/gstlal_peakfinder.c b/gstlal/lib/gstlal/gstlal_peakfinder.c
index 1a1d83de5b..477493e196 100644
--- a/gstlal/lib/gstlal/gstlal_peakfinder.c
+++ b/gstlal/lib/gstlal/gstlal_peakfinder.c
@@ -213,7 +213,7 @@ int gstlal_peak_max_over_channels(struct gstlal_peak_state *state)
 			{
 				if(fabsf(state->values.as_float[i]) > max_val)
 				{
-					max_val = state->values.as_float[i];
+					max_val = fabsf(state->values.as_float[i]);
 					out = i;
 				}
 			}
@@ -228,7 +228,7 @@ int gstlal_peak_max_over_channels(struct gstlal_peak_state *state)
 			{
 				if(fabs(state->values.as_double[i]) > max_val)
 				{
-					max_val = state->values.as_double[i];
+					max_val = fabs(state->values.as_double[i]);
 					out = i;
 				}
 			}
@@ -243,7 +243,7 @@ int gstlal_peak_max_over_channels(struct gstlal_peak_state *state)
 			{
 				if(cabsf(state->values.as_float_complex[i]) > max_val)
 				{
-					max_val = state->values.as_float_complex[i];
+					max_val = cabsf(state->values.as_float_complex[i]);
 					out = i;
 				}
 			}
@@ -258,7 +258,7 @@ int gstlal_peak_max_over_channels(struct gstlal_peak_state *state)
 			{
 				if(cabs(state->values.as_double_complex[i]) > max_val)
 				{
-					max_val = state->values.as_double_complex[i];
+					max_val = cabs(state->values.as_double_complex[i]);
 					out = i;
 				}
 			}
-- 
GitLab