From e820814809f6bef69f1501f16fb797d841034dec Mon Sep 17 00:00:00 2001
From: Matthias <xmatthias@outlook.com>
Date: Wed, 14 Apr 2021 20:32:34 +0200
Subject: [PATCH] Default-stoploss-hyperopt should use decimal space, nto real

---
 freqtrade/optimize/hyperopt_interface.py        | 3 ++-
 freqtrade/optimize/space/__init__.py            | 4 ++++
 freqtrade/optimize/{ => space}/decimalspace.py  | 0
 freqtrade/strategy/hyper.py                     | 2 +-
 freqtrade/templates/sample_hyperopt_advanced.py | 4 ++--
 tests/optimize/test_hyperopt.py                 | 2 +-
 6 files changed, 10 insertions(+), 5 deletions(-)
 create mode 100644 freqtrade/optimize/space/__init__.py
 rename freqtrade/optimize/{ => space}/decimalspace.py (100%)

diff --git a/freqtrade/optimize/hyperopt_interface.py b/freqtrade/optimize/hyperopt_interface.py
index 633c8bdd5..fa28463e9 100644
--- a/freqtrade/optimize/hyperopt_interface.py
+++ b/freqtrade/optimize/hyperopt_interface.py
@@ -12,6 +12,7 @@ from skopt.space import Categorical, Dimension, Integer, Real
 from freqtrade.exceptions import OperationalException
 from freqtrade.exchange import timeframe_to_minutes
 from freqtrade.misc import round_dict
+from freqtrade.optimize.space import SKDecimal
 from freqtrade.strategy import IStrategy
 
 
@@ -167,7 +168,7 @@ class IHyperOpt(ABC):
         You may override it in your custom Hyperopt class.
         """
         return [
-            Real(-0.35, -0.02, name='stoploss'),
+            SKDecimal(-0.35, -0.02, decimals=3, name='stoploss'),
         ]
 
     def generate_trailing_params(self, params: Dict) -> Dict:
diff --git a/freqtrade/optimize/space/__init__.py b/freqtrade/optimize/space/__init__.py
new file mode 100644
index 000000000..bbdac4ab9
--- /dev/null
+++ b/freqtrade/optimize/space/__init__.py
@@ -0,0 +1,4 @@
+# flake8: noqa: F401
+from skopt.space import Categorical, Dimension, Integer, Real
+
+from .decimalspace import SKDecimal
diff --git a/freqtrade/optimize/decimalspace.py b/freqtrade/optimize/space/decimalspace.py
similarity index 100%
rename from freqtrade/optimize/decimalspace.py
rename to freqtrade/optimize/space/decimalspace.py
diff --git a/freqtrade/strategy/hyper.py b/freqtrade/strategy/hyper.py
index 3fedda974..16b576a73 100644
--- a/freqtrade/strategy/hyper.py
+++ b/freqtrade/strategy/hyper.py
@@ -10,7 +10,7 @@ from typing import Any, Iterator, Optional, Sequence, Tuple, Union
 
 with suppress(ImportError):
     from skopt.space import Integer, Real, Categorical
-    from freqtrade.optimize.decimalspace import SKDecimal
+    from freqtrade.optimize.space import SKDecimal
 
 from freqtrade.exceptions import OperationalException
 
diff --git a/freqtrade/templates/sample_hyperopt_advanced.py b/freqtrade/templates/sample_hyperopt_advanced.py
index 7736570f7..32ba21716 100644
--- a/freqtrade/templates/sample_hyperopt_advanced.py
+++ b/freqtrade/templates/sample_hyperopt_advanced.py
@@ -7,7 +7,7 @@ from typing import Any, Callable, Dict, List
 import numpy as np  # noqa
 import pandas as pd  # noqa
 from pandas import DataFrame
-from skopt.space import Categorical, Dimension, Integer, Real  # noqa
+from freqtrade.optimize.space import Categorical, Dimension, Integer, SKDecimal, Real  # noqa
 
 from freqtrade.optimize.hyperopt_interface import IHyperOpt
 
@@ -237,7 +237,7 @@ class AdvancedSampleHyperOpt(IHyperOpt):
         'stoploss' optimization hyperspace.
         """
         return [
-            Real(-0.35, -0.02, name='stoploss'),
+            SKDecimal(-0.35, -0.02, decimals=3, name='stoploss'),
         ]
 
     @staticmethod
diff --git a/tests/optimize/test_hyperopt.py b/tests/optimize/test_hyperopt.py
index 129fe53d9..59bc4aefb 100644
--- a/tests/optimize/test_hyperopt.py
+++ b/tests/optimize/test_hyperopt.py
@@ -15,10 +15,10 @@ from filelock import Timeout
 from freqtrade.commands.optimize_commands import setup_optimize_configuration, start_hyperopt
 from freqtrade.data.history import load_data
 from freqtrade.exceptions import OperationalException
-from freqtrade.optimize.decimalspace import SKDecimal
 from freqtrade.optimize.hyperopt import Hyperopt
 from freqtrade.optimize.hyperopt_auto import HyperOptAuto
 from freqtrade.optimize.hyperopt_tools import HyperoptTools
+from freqtrade.optimize.space import SKDecimal
 from freqtrade.resolvers.hyperopt_resolver import HyperOptResolver
 from freqtrade.state import RunMode
 from tests.conftest import (get_args, log_has, log_has_re, patch_exchange,