From 8a56db8168cbbcace4cd55a1cc1b22528dcde5c8 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=D0=9F=D0=B0=D0=B2=D0=B5=D0=BB=20=D0=96=D1=83=D0=BA=D0=BE?= =?UTF-8?q?=D0=B2?= <33721692+LeaveMyYard@users.noreply.github.com> Date: Tue, 28 Mar 2023 19:26:05 +0300 Subject: Fix invalid calculation in simple strategy --- robusta_krr/strategies/simple.py | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/robusta_krr/strategies/simple.py b/robusta_krr/strategies/simple.py index 1846049..8206227 100644 --- a/robusta_krr/strategies/simple.py +++ b/robusta_krr/strategies/simple.py @@ -37,10 +37,4 @@ class SimpleStrategy(BaseStrategy[SimpleStrategySettings]): if len(data_) == 0: return Decimal.from_float(float("nan")) - min_val, max_val = min(data_), max(data_) - # If the min and max are the same, we can't calculate a percentile - # Should'n happen on practice, but just in case - if min_val == max_val: - return min_val - - return min_val + (max_val - min_val) * percentile / 100 + return max(data_) * percentile / 100 -- cgit v1.2.3