diff --git a/freqtrade/data/history.py b/freqtrade/data/history.py index a71dc373c..b4776fab0 100644 --- a/freqtrade/data/history.py +++ b/freqtrade/data/history.py @@ -61,7 +61,7 @@ def load_tickerdata_file(datadir: Path, pair: str, ticker_interval: str, timerange: Optional[TimeRange] = None) -> Optional[list]: """ Load a pair from file, either .json.gz or .json - :return: tickerlist or None if unsuccesful + :return: tickerlist or None if unsuccessful """ filename = pair_data_filename(datadir, pair, ticker_interval) pairdata = misc.file_load_json(filename) diff --git a/freqtrade/plot/plot_utils.py b/freqtrade/plot/plot_utils.py index d7d8919d5..d7fb326d1 100644 --- a/freqtrade/plot/plot_utils.py +++ b/freqtrade/plot/plot_utils.py @@ -17,11 +17,11 @@ def start_plot_dataframe(args: Namespace) -> None: Entrypoint for dataframe plotting """ # Import here to avoid errors if plot-dependencies are not installed. - from freqtrade.plot.plotting import analyse_and_plot_pairs + from freqtrade.plot.plotting import load_and_plot_trades validate_plot_args(args) config = setup_utils_configuration(args, RunMode.PLOT) - analyse_and_plot_pairs(config) + load_and_plot_trades(config) def start_plot_profit(args: Namespace) -> None: diff --git a/freqtrade/plot/plotting.py b/freqtrade/plot/plotting.py index b0b8e3df9..1627959f9 100644 --- a/freqtrade/plot/plotting.py +++ b/freqtrade/plot/plotting.py @@ -3,7 +3,6 @@ from pathlib import Path from typing import Any, Dict, List import pandas as pd - from freqtrade.configuration import TimeRange from freqtrade.data import history from freqtrade.data.btanalysis import (combine_tickers_with_mean, @@ -324,7 +323,7 @@ def store_plot_file(fig, filename: str, directory: Path, auto_open: bool = False logger.info(f"Stored plot as {_filename}") -def analyse_and_plot_pairs(config: Dict[str, Any]): +def load_and_plot_trades(config: Dict[str, Any]): """ From configuration provided - Initializes plot-script @@ -339,7 +338,6 @@ def analyse_and_plot_pairs(config: Dict[str, Any]): plot_elements = init_plotscript(config) trades = plot_elements['trades'] - pair_counter = 0 for pair, data in plot_elements["tickers"].items(): pair_counter += 1 @@ -348,7 +346,6 @@ def analyse_and_plot_pairs(config: Dict[str, Any]): tickers[pair] = data dataframe = strategy.analyze_ticker(tickers[pair], {'pair': pair}) - trades_pair = trades.loc[trades['pair'] == pair] trades_pair = extract_trades_of_period(dataframe, trades_pair) diff --git a/tests/test_plotting.py b/tests/test_plotting.py index a30d7d04a..7debff071 100644 --- a/tests/test_plotting.py +++ b/tests/test_plotting.py @@ -32,7 +32,7 @@ def find_trace_in_fig_data(data, search_string: str): return next(matches) -def generage_empty_figure(): +def generate_empty_figure(): return make_subplots( rows=3, cols=1, @@ -72,7 +72,7 @@ def test_add_indicators(default_conf, testdatadir, caplog): # Generate buy/sell signals and indicators strat = DefaultStrategy(default_conf) data = strat.analyze_ticker(data, {'pair': pair}) - fig = generage_empty_figure() + fig = generate_empty_figure() # Row 1 fig1 = add_indicators(fig=deepcopy(fig), row=1, indicators=indicators1, data=data) @@ -210,7 +210,7 @@ def test_generate_Plot_filename(): def test_generate_plot_file(mocker, caplog): - fig = generage_empty_figure() + fig = generate_empty_figure() plot_mock = mocker.patch("freqtrade.plot.plotting.plot", MagicMock()) store_plot_file(fig, filename="freqtrade-plot-UNITTEST_BTC-5m.html", directory=Path("user_data/plots"))