diff --git a/freqtrade/freqai/freqai_interface.py b/freqtrade/freqai/freqai_interface.py index 7f2fd677c..55f5a03a0 100644 --- a/freqtrade/freqai/freqai_interface.py +++ b/freqtrade/freqai/freqai_interface.py @@ -511,7 +511,7 @@ class IFreqaiModel(ABC): """ @abstractmethod - def fit(self) -> Any: + def fit(self, data_dictionary: Dict[str, Any]) -> Any: """ Most regressors use the same function names and arguments e.g. user can drop in LGBMRegressor in place of CatBoostRegressor and all data diff --git a/freqtrade/resolvers/freqaimodel_resolver.py b/freqtrade/resolvers/freqaimodel_resolver.py index 0fcfca363..5a847bb2b 100644 --- a/freqtrade/resolvers/freqaimodel_resolver.py +++ b/freqtrade/resolvers/freqaimodel_resolver.py @@ -44,7 +44,7 @@ class FreqaiModelResolver(IResolver): ) if freqaimodel_name in disallowed_models: raise OperationalException( - f"{freqaimodel_name} is a baseclass and cannot be used directly. User must choose " + f"{freqaimodel_name} is a baseclass and cannot be used directly. Please choose " "an existing child class or inherit from this baseclass.\n" ) freqaimodel = FreqaiModelResolver.load_object( diff --git a/freqtrade/strategy/interface.py b/freqtrade/strategy/interface.py index ed458786b..df73981bd 100644 --- a/freqtrade/strategy/interface.py +++ b/freqtrade/strategy/interface.py @@ -555,7 +555,8 @@ class IStrategy(ABC, HyperStrategyMixin): Function designed to automatically generate, name and merge features from user indicated timeframes in the configuration file. User can add additional features here, but must follow the naming convention. - Defined in IStrategy because Freqai needs to know it exists. + This method is *only* used in FreqaiDataKitchen class and therefore + it is only called if FreqAI is active. :params: :pair: pair to be used as informative :df: strategy dataframe which will receive merges from informatives