From 3db05d081733364d16f5b7e83bcb7a023490f406 Mon Sep 17 00:00:00 2001 From: Matthias Date: Tue, 8 Mar 2022 19:12:17 +0100 Subject: [PATCH] Simplify Hourbased strategy by not using intermediate min/max variables --- user_data/strategies/HourBasedStrategy.py | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/user_data/strategies/HourBasedStrategy.py b/user_data/strategies/HourBasedStrategy.py index 937f4c5..b38be74 100644 --- a/user_data/strategies/HourBasedStrategy.py +++ b/user_data/strategies/HourBasedStrategy.py @@ -86,20 +86,18 @@ class HourBasedStrategy(IStrategy): return dataframe def populate_buy_trend(self, dataframe: DataFrame, metadata: dict) -> DataFrame: - min, max = self.buy_hour_min.value, self.buy_hour_max.value dataframe.loc[ ( - (dataframe['hour'].between(min, max)) + (dataframe['hour'].between(self.buy_hour_min.value, self.buy_hour_max.value)) ), 'buy'] = 1 return dataframe def populate_sell_trend(self, dataframe: DataFrame, metadata: dict) -> DataFrame: - min, max = self.sell_hour_min.value, self.sell_hour_max.value dataframe.loc[ ( - (dataframe['hour'].between(max, min)) + (dataframe['hour'].between(self.sell_hour_min.value, self.sell_hour_max.value)) ), 'sell'] = 1 return dataframe