Repository: freqtrade/freqtrade

Issue:
Hyperopt confusing warning message
## Step 1: Describe your environment

  * Operating system: Windows 10
  * Python Version: 3.7.5 (`python -V`)
  * Branch: freqtrade develop-1aa1fada
  
## Step 2: Describe the problem:

When hyperopting with the following command: `freqtrade hyperopt -c test_config.json --hyperopt strategy_hyperopt -e 750--spaces buy sell stoploss trailing -–min-trades=20 –-timerange=20200111-20200510 –j 2 --no-color` 

I get the following warning messages:
![image](https://user-images.githubusercontent.com/24569139/82764678-db0dc700-9e10-11ea-9679-a9efc64f6207.png)

I didn't include the 'normal' buy/sell trends in the hyperopt file, but since I am using the buy/sell trends generated by the hyperopt file, these messages can be confusing. It's taking the 'normal' buy/sell trends from the original strategy file, but it doesn't use it anyway.



Discussion:
well it depends on the space you're using if this method is used or not.

Assume you use `--spaces buy` - then the `populate_sell` would be used.

I guess it could be just an info message - but it's there to let you know that the hyperopt file does not contain `populate_buy` and `populate_sell` methods - which might seem unnecessary if you know you left them out - but can be very helpful if you have a typo in the function name.
buy and sell spaces are specified -- in this case only decorated methods from hyperopt class are used, hyperopt should not issue a message about the missing non-decorated methods.

i disagree - it'll make spotting "typos" very difficult - where the wrong method is being used due to a typo in the header.

Keep in mind that these were added to prevent this kind of problem searching.

It might not be worth a "Warning" - but an info level instead ... but the message telling you that hyperopt does NOT contain these methods should remain.
okay, maybe info
or maybe still a warning in this case, to have it more visible,
need to think about this a bit...

Resolving patch:
diff --git a/freqtrade/resolvers/hyperopt_resolver.py b/freqtrade/resolvers/hyperopt_resolver.py
--- a/freqtrade/resolvers/hyperopt_resolver.py
+++ b/freqtrade/resolvers/hyperopt_resolver.py
@@ -42,14 +42,14 @@ def load_hyperopt(config: Dict) -> IHyperOpt:
                                                 extra_dir=config.get('hyperopt_path'))
 
         if not hasattr(hyperopt, 'populate_indicators'):
-            logger.warning("Hyperopt class does not provide populate_indicators() method. "
-                           "Using populate_indicators from the strategy.")
+            logger.info("Hyperopt class does not provide populate_indicators() method. "
+                        "Using populate_indicators from the strategy.")
         if not hasattr(hyperopt, 'populate_buy_trend'):
-            logger.warning("Hyperopt class does not provide populate_buy_trend() method. "
-                           "Using populate_buy_trend from the strategy.")
+            logger.info("Hyperopt class does not provide populate_buy_trend() method. "
+                        "Using populate_buy_trend from the strategy.")
         if not hasattr(hyperopt, 'populate_sell_trend'):
-            logger.warning("Hyperopt class does not provide populate_sell_trend() method. "
-                           "Using populate_sell_trend from the strategy.")
+            logger.info("Hyperopt class does not provide populate_sell_trend() method. "
+                        "Using populate_sell_trend from the strategy.")
         return hyperopt
 
 


Test patch: