From 2fb5315673abebe26debcaa0e05770e2f471d640 Mon Sep 17 00:00:00 2001
From: Lionel GUEZ <guez@lmd.ens.fr>
Date: Wed, 29 May 2024 09:41:14 +0200
Subject: [PATCH] Define intermediary variable `i_max`

---
 Trajectories/cost_function.py | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/Trajectories/cost_function.py b/Trajectories/cost_function.py
index b795e0dd..c62860cb 100755
--- a/Trajectories/cost_function.py
+++ b/Trajectories/cost_function.py
@@ -150,9 +150,10 @@ def search_end(inst_eddies, max_delta, avg_fix, e_overestim):
     computation is not implemented that way.
 
     """
-    ip = max(len(inst_eddies) - max_delta - 1, 0)
+    i_max = len(inst_eddies) - 1
+    ip = max(i_max - max_delta, 0)
 
-    if not avg_fix and ip <= len(inst_eddies) - 2:
+    if not avg_fix and ip < i_max:
         d_min = (
             util_eddies.node_to_date_eddy(
                 inst_eddies[-1], e_overestim, only_date=True
@@ -162,7 +163,7 @@ def search_end(inst_eddies, max_delta, avg_fix, e_overestim):
         # {date(elem) < d_min for elem in inst_eddies[:ip]}
 
         while (
-            ip <= len(inst_eddies) - 2
+            ip < i_max
             and util_eddies.node_to_date_eddy(
                 inst_eddies[ip], e_overestim, only_date=True
             )
-- 
GitLab