🚧 make outputdir option for streamlit optional, move to init

- outputdir only really makes sense for report generation,
  not execution: In the current implementation the report has to execute from the path it was generated from
- should the static dir be change upon change of section_dir?

Author: enryH

src/vuegen/report_generator.py

@@ -108,9 +108,10 @@ def get_report(
             report_type=report_type,
             streamlit_autorun=streamlit_autorun,
             static_dir=static_files_dir,
+            sections_dir=sections_dir,
         )
-        st_report.generate_report(output_dir=sections_dir)
-        st_report.run_report(output_dir=sections_dir)
+        st_report.generate_report()
+        st_report.run_report()
     else:
         # Check if Quarto is installed
         if shutil.which("quarto") is None and not hasattr(

src/vuegen/streamlit_reportview.py

@@ -50,6 +50,7 @@ def __init__(
         report_type: r.ReportType,
         streamlit_autorun: bool = False,
         static_dir: str = STATIC_FILES_DIR,
+        sections_dir: str = SECTIONS_DIR,
     ):
         """Initialize ReportView with the report and report type.
 
@@ -86,8 +87,9 @@ def __init__(
         }
 
         self.static_dir = static_dir
+        self.section_dir = sections_dir
 
-    def generate_report(self, output_dir: str = SECTIONS_DIR) -> None:
+    def generate_report(self, output_dir: str = None) -> None:
         """
         Generates the Streamlit report and creates Python files for each section
         and its subsections and plots.
@@ -98,6 +100,10 @@ def generate_report(self, output_dir: str = SECTIONS_DIR) -> None:
             The folder where the generated report files will be saved
             (default is SECTIONS_DIR).
         """
+        if output_dir is not None:
+            # ? does this imply changes to the static dir
+            self.section_dir = Path(output_dir).resolve()
+        output_dir = Path(self.section_dir)
         self.report.logger.debug(
             "Generating '%s' report in directory: '%s'", self.report_type, output_dir
         )
@@ -282,7 +288,7 @@ def generate_report(self, output_dir: str = SECTIONS_DIR) -> None:
             )
             raise
 
-    def run_report(self, output_dir: str = SECTIONS_DIR) -> None:
+    def run_report(self, output_dir: str = None) -> None:
         """
         Runs the generated Streamlit report.
 
@@ -291,6 +297,9 @@ def run_report(self, output_dir: str = SECTIONS_DIR) -> None:
         output_dir : str, optional
             The folder where the report was generated (default is SECTIONS_DIR).
         """
+        if output_dir is not None:
+            self.report.logger.warning("The output_dir parameter is deprecated.")
+        output_dir = Path(self.section_dir)
         if self.streamlit_autorun:
             self.report.logger.info(
                 "Running '%s' %s report.", self.report.title, self.report_type