From b944b9e23b3a42d24c9b974f4c4e1f50a72846ad Mon Sep 17 00:00:00 2001
From: Shane Smiskol <shane@smiskol.com>
Date: Mon, 28 Jul 2025 19:45:59 -0700
Subject: [PATCH] rm it

---
 common/filter_simple.py |  8 ++++----
 estimate_lat_jerk.py    | 24 ++++++++++--------------
 2 files changed, 14 insertions(+), 18 deletions(-)

diff --git a/common/filter_simple.py b/common/filter_simple.py
index c7c63241d1..ae542bc78b 100644
--- a/common/filter_simple.py
+++ b/common/filter_simple.py
@@ -17,7 +17,7 @@ class FirstOrderFilter:
     return self.x
 
 
-class JerkEstimator2:
+class JerkEstimator1:
   def __init__(self, dt):
     self.dt = dt
     self.prev_x = 0.0
@@ -38,7 +38,7 @@ class JerkEstimator2:
     return self.filter.x
 
 
-class JerkEstimator3:
+class JerkEstimator2:
   def __init__(self, dt):
     self.dt = dt
     self.initialized = False
@@ -67,7 +67,7 @@ class JerkEstimator3:
     return self.kf.x[1][0]
 
 
-class JerkEstimator4:
+class JerkEstimator3:
   def __init__(self, dt):
     self.dt = dt
     self.prev_x = 0.0
@@ -92,7 +92,7 @@ class JerkEstimator4:
     return self._x
 
 
-class JerkEstimator5:
+class JerkEstimator4:
   def __init__(self, dt):
     from collections import deque
     self.dt = dt
diff --git a/estimate_lat_jerk.py b/estimate_lat_jerk.py
index 05a7fc9b5e..cf546a42dd 100644
--- a/estimate_lat_jerk.py
+++ b/estimate_lat_jerk.py
@@ -1,6 +1,6 @@
 import math
 import matplotlib.pyplot as plt
-from openpilot.common.filter_simple import JerkEstimator2, JerkEstimator3, JerkEstimator4, JerkEstimator5
+from openpilot.common.filter_simple import JerkEstimator1, JerkEstimator2, JerkEstimator3, JerkEstimator4
 from tools.lib.logreader import LogReader
 from openpilot.selfdrive.locationd.helpers import PoseCalibrator, Pose
 from opendbc.car import ACCELERATION_DUE_TO_GRAVITY
@@ -13,14 +13,13 @@ calibrator = PoseCalibrator()
 sm = {}
 
 j1 = JerkEstimator1(1/20)
-j2 = JerkEstimator2(1/20)
-j3 = JerkEstimator3(1/100)
+j2 = JerkEstimator2(1/100)
+j3 = JerkEstimator3(1/20)
 j4 = JerkEstimator4(1/20)
-j5 = JerkEstimator5(1/20)
 
 accels = []
 kf_accels = []
-jerks1, jerks2, jerks3, jerks4, jerks5 = [], [], [], [], []
+jerks1, jerks2, jerks3, jerks4 = [], [], [], []
 lp_updated = False
 
 for msg in lr:
@@ -41,24 +40,22 @@ for msg in lr:
     roll = sm['liveParameters'].roll
     roll_compensated_lateral_accel = (CS.vEgo * yaw_rate) - (math.sin(roll) * ACCELERATION_DUE_TO_GRAVITY)
 
-    _j3 = j3.update(roll_compensated_lateral_accel)
+    _j3 = j2.update(roll_compensated_lateral_accel)
     if lp_updated:
       _j1 = j1.update(roll_compensated_lateral_accel)
       _j2 = j2.update(roll_compensated_lateral_accel)
-      _j4 = j4.update(roll_compensated_lateral_accel)
-      _j5 = j5.update(roll_compensated_lateral_accel)
+      _j4 = j3.update(roll_compensated_lateral_accel)
       lp_updated = False
     else:
       _j1 = j1.x
       _j2 = j2.x
+      _j3 = j3.x
       _j4 = j4.x
-      _j5 = j5.x
 
     jerks1.append(_j1)
     jerks2.append(_j2)
     jerks3.append(_j3)
     jerks4.append(_j4)
-    jerks5.append(_j5)
     accels.append(roll_compensated_lateral_accel)
 
     print(roll_compensated_lateral_accel)
@@ -71,10 +68,9 @@ axs[0].set_ylabel('Lateral Acceleration (m/s²)')
 axs[0].legend()
 
 axs[1].plot(jerks1, label='Jerk Estimator 1')
-axs[1].plot(jerks2, label='Jerk Estimator 2')
-# axs[1].plot(jerks3, label='Jerk Estimator 3')
-# axs[1].plot(jerks4, label='Jerk Estimator 4')
-# axs[1].plot(jerks5, label='Jerk Estimator 5')
+# axs[1].plot(jerks2, label='Jerk Estimator 2')
+axs[1].plot(jerks3, label='Jerk Estimator 3')
+axs[1].plot(jerks4, label='Jerk Estimator 4')
 axs[1].set_ylabel('Lateral Jerk (m/s³)')
 axs[1].legend()