mohammadian7/freqtrade
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

chore: clarify --export-filename Deprecation messages

Browse Source 
part of #12846
This commit is contained in:
Matthias
2026-02-21 20:20:14 +01:00
parent a417af1ff0
commit ce590cced7
2 changed files with 18 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files
freqtrade/configuration/configuration.py
+15 -8
View File
@@ -221,14 +221,21 @@ class Configuration:
config, argname="exportfilename", logstring="Storing backtest results to {} ..." config, argname="exportfilename", logstring="Storing backtest results to {} ..."
) )
config["exportfilename"] = Path(config["exportfilename"]) config["exportfilename"] = Path(config["exportfilename"])
if config.get("exportdirectory") and Path(config["exportdirectory"]).is_dir(): if config.get("exportfilename"):
logger.warning( if Path(config["exportfilename"]).is_dir():
"DEPRECATED: Using `--export-filename` with directories is deprecated, " logger.warning(
"use `--backtest-directory` instead." "DEPRECATED: Using `--export-filename` with directories is deprecated, "
) "use `--backtest-directory` instead."
if config.get("exportdirectory") is None: )
# Fallback - assign export-directory directly. if config.get("exportdirectory") is None:
config["exportdirectory"] = config["exportfilename"] # Fallback - assign export-directory directly.
config["exportdirectory"] = config["exportfilename"]
elif config.get("runmode") == RunMode.BACKTEST:
logger.warning(
"DEPRECATED: Using `--export-filename` has no impact when backtesting. "
"Please use `--notes` to annotate backtest results and "
"`--backtest-directory` to specify the output directory. "
)
if not config.get("exportdirectory"): if not config.get("exportdirectory"):
config["exportdirectory"] = config["user_data_dir"] / "backtest_results" config["exportdirectory"] = config["user_data_dir"] / "backtest_results"
if not config.get("exportfilename"): if not config.get("exportfilename"):
tests/optimize/test_backtesting.py
+3
View File
@@ -221,6 +221,9 @@ def test_setup_bt_configuration_with_arguments(mocker, default_conf, caplog) ->
assert "exportfilename" in config assert "exportfilename" in config
assert isinstance(config["exportfilename"], Path) assert isinstance(config["exportfilename"], Path)
assert log_has("Storing backtest results to {} ...".format(config["exportfilename"]), caplog) assert log_has("Storing backtest results to {} ...".format(config["exportfilename"]), caplog)
assert log_has_re(
"DEPRECATED: Using `--export-filename` has no impact when backtesting.*", caplog
)
assert "fee" in config assert "fee" in config
assert log_has("Parameter --fee detected, setting fee to: {} ...".format(config["fee"]), caplog) assert log_has("Parameter --fee detected, setting fee to: {} ...".format(config["fee"]), caplog)