small comment on show() implementation

OpenBB-finance · Jul 28, 2023 · 379a551 · 379a551
1 parent 849e613
commit 379a551
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/openbb_sdk/sdk/core/openbb_core/app/model/charts/chart.py b/openbb_sdk/sdk/core/openbb_core/app/model/charts/chart.py
@@ -20,6 +20,9 @@ class Config:
     def show(self):
         """Shows the chart in PyWry, browser or notebook."""
 
+        # TODO : this method should be handled by the ChartingManager so that
+        # this model class doesn't have specific knowledge on the charting extension
+
         # pylint: disable=import-outside-toplevel
         if self.format == ChartFormat.plotly:
             from openbb_charting.core.openbb_figure import OpenBBFigure