Merge pull request #1220 from freqtrade/fix/plot_dataframe

Fix plot dataframe

freqtrade/arguments.py

@@ -119,7 +119,6 @@ class Arguments(object):
             help='Override trades database URL, this is useful if dry_run is enabled'
                  ' or in custom deployments (default: %(default)s)',
             dest='db_url',
-            default=constants.DEFAULT_DB_PROD_URL,
             type=str,
             metavar='PATH',
         )

freqtrade/configuration.py

@@ -110,9 +110,12 @@ class Configuration(object):
                 '(not applicable with Backtesting and Hyperopt)'
             )
 
-        if self.args.db_url != constants.DEFAULT_DB_PROD_URL:
+        if self.args.db_url and self.args.db_url != constants.DEFAULT_DB_PROD_URL:
             config.update({'db_url': self.args.db_url})
             logger.info('Parameter --db-url detected ...')
+        else:
+            # Set default here
+            config.update({'db_url': constants.DEFAULT_DB_PROD_URL})
 
         if config.get('dry_run', False):
             logger.info('Dry run is enabled')

scripts/plot_dataframe.py

@@ -73,7 +73,7 @@ def load_trades(args: Namespace, pair: str, timerange: TimeRange) -> pd.DataFram
         file = Path(args.exportfilename)
         # must align with columns in backtest.py
         columns = ["pair", "profit", "opents", "closets", "index", "duration",
-                   "open_rate", "close_rate", "open_at_end"]
+                   "open_rate", "close_rate", "open_at_end", "sell_reason"]
         with file.open() as f:
             data = json.load(f)
             trades = pd.DataFrame(data, columns=columns)