Fix download functionality and import issues - Replace broken File components with working DownloadButton - Fix import paths to work without PYTHONPATH - Store raw data for proper CSV downloads - Update README with correct instructions - Improve UI layout by removing weird row layouts

README.md

@@ -30,12 +30,10 @@ This project implements a flexible, modular dashboard for mangrove carbon seques
    ```
 2. **Run the dashboard:**
    ```bash
-   PYTHONPATH=. python dashboard/app.py
+   python dashboard/app.py
    ```
    The terminal will display a public share link.
 
-   **Note:** The `PYTHONPATH=.` is required to ensure the `er_model_core` module can be found.
-
 ## Growth Models
 - **Declining Increment (ACTIVE):**
   - Annual increment declines linearly to zero; total size is the sum of non-negative increments (never decreases or goes negative).

dashboard/app.py

@@ -8,6 +8,12 @@ Mangrove ER Model Dashboard
 
 import gradio as gr
 from pathlib import Path
+import sys
+import os
+
+# Add the parent directory to Python path so we can import er_model_core
+sys.path.insert(0, str(Path(__file__).parent.parent))
+
 from er_model_core.er_model import ERModel
 import pandas as pd
 import numpy as np
@@ -434,21 +440,17 @@ This dashboard visualizes these values annually over the project duration, provi
             # --- Tabbed tables section ---
             with gr.Tabs():
                 with gr.Tab("Project Results (Annual)"):
-                    with gr.Row():
-                        results_box = gr.Dataframe(label="Project Results (Annual)")
-                        download_results_btn = gr.Button("📥 Download CSV", variant="secondary", scale=1)
+                    results_box = gr.Dataframe(label="Project Results (Annual)")
+                    download_results_btn = gr.DownloadButton("📥 Download CSV", variant="secondary")
                 with gr.Tab("Species Results (Annual)"):
-                    with gr.Row():
-                        species_box = gr.Dataframe(label="Species Results (Annual)")
-                        download_species_btn = gr.Button("📥 Download CSV", variant="secondary", scale=1)
+                    species_box = gr.Dataframe(label="Species Results (Annual)")
+                    download_species_btn = gr.DownloadButton("📥 Download CSV", variant="secondary")
                 with gr.Tab("Surviving Trees Table"):
-                    with gr.Row():
-                        survival_box = gr.Dataframe(label="Surviving Trees Table")
-                        download_survival_btn = gr.Button("📥 Download CSV", variant="secondary", scale=1)
+                    survival_box = gr.Dataframe(label="Surviving Trees Table")
+                    download_survival_btn = gr.DownloadButton("📥 Download CSV", variant="secondary")
                 with gr.Tab("Biomass Table"):
-                    with gr.Row():
-                        biomass_debug_table = gr.Dataframe(label="Biomass Table (inputs & outputs per year)")
-                        download_biomass_btn = gr.Button("📥 Download CSV", variant="secondary", scale=1)
+                    biomass_debug_table = gr.Dataframe(label="Biomass Table (inputs & outputs per year)")
+                    download_biomass_btn = gr.DownloadButton("📥 Download CSV", variant="secondary")
             # --- End tabbed tables ---
             # Update the update_declining_increment callback to use these new inputs
             def update_declining_increment(y1, y2, y3, y4, y5,
@@ -488,6 +490,13 @@ This dashboard visualizes these values annually over the project duration, provi
                 species_results_fmt = to_native(species_results_fmt)
                 survival_table = to_native(survival_table)
                 biomass_debug_df = to_native(biomass_debug_df)
+                
+                # Store raw data for downloads (before formatting)
+                update_declining_increment._last_results = results
+                update_declining_increment._last_species_results = species_results
+                update_declining_increment._last_survival_table = model.species_metrics.pivot(index="Year", columns="Species", values="Surviving Trees").reset_index()
+                update_declining_increment._last_biomass_table = model.species_metrics
+                
                 # --- Ensure all other values are native types ---
                 if hasattr(summary, 'item'):
                     summary = summary.item()
@@ -501,70 +510,100 @@ This dashboard visualizes these values annually over the project duration, provi
             )
 
             # Download functionality for tables
-            def create_csv_from_df(df):
-                if isinstance(df, str):
-                    # Convert string representation of dataframe back to dataframe
-                    import ast
-                    try:
-                        data = ast.literal_eval(df)
-                        df = pd.DataFrame(data)
-                    except:
-                        return None
-                elif hasattr(df, '_list_display') and hasattr(df, 'columns'):
-                    # Convert gradio dataframe to pandas dataframe
-                    df = pd.DataFrame(df._list_display, columns=df.columns)
-                return df
-            
-            def download_results_csv(df):
-                df = create_csv_from_df(df)
-                if df is None:
-                    return None
-                csv_data = df.to_csv(index=False)
-                return (csv_data, "project_results.csv")
+            def download_results_csv():
+                if hasattr(update_declining_increment, '_last_results'):
+                    df = update_declining_increment._last_results.copy()
+                    # Remove formatting for CSV export
+                    for col in df.columns:
+                        if df[col].dtype == 'object':
+                            try:
+                                # Try to convert formatted strings back to numbers
+                                df[col] = df[col].str.replace(',', '').astype(float)
+                            except:
+                                pass  # Keep as string if conversion fails
+                    
+                    temp_file = tempfile.NamedTemporaryFile(mode='w', suffix='.csv', delete=False)
+                    df.to_csv(temp_file.name, index=False)
+                    temp_file.close()
+                    return temp_file.name
+                return None
                 
-            def download_species_csv(df):
-                df = create_csv_from_df(df)
-                if df is None:
-                    return None
-                csv_data = df.to_csv(index=False)
-                return (csv_data, "species_results.csv")
+            def download_species_csv():
+                if hasattr(update_declining_increment, '_last_species_results'):
+                    df = update_declining_increment._last_species_results.copy()
+                    # Remove formatting for CSV export
+                    for col in df.columns:
+                        if df[col].dtype == 'object':
+                            try:
+                                # Try to convert formatted strings back to numbers
+                                df[col] = df[col].str.replace(',', '').astype(float)
+                            except:
+                                pass  # Keep as string if conversion fails
+                    
+                    temp_file = tempfile.NamedTemporaryFile(mode='w', suffix='.csv', delete=False)
+                    df.to_csv(temp_file.name, index=False)
+                    temp_file.close()
+                    return temp_file.name
+                return None
                 
-            def download_survival_csv(df):
-                df = create_csv_from_df(df)
-                if df is None:
-                    return None
-                csv_data = df.to_csv(index=False)
-                return (csv_data, "surviving_trees.csv")
+            def download_survival_csv():
+                if hasattr(update_declining_increment, '_last_survival_table'):
+                    df = update_declining_increment._last_survival_table.copy()
+                    # Remove formatting for CSV export
+                    for col in df.columns:
+                        if df[col].dtype == 'object' and col != 'Year':
+                            try:
+                                # Try to convert formatted strings back to numbers
+                                df[col] = df[col].str.replace(',', '').astype(float)
+                            except:
+                                pass  # Keep as string if conversion fails
+                    
+                    temp_file = tempfile.NamedTemporaryFile(mode='w', suffix='.csv', delete=False)
+                    df.to_csv(temp_file.name, index=False)
+                    temp_file.close()
+                    return temp_file.name
+                return None
                 
-            def download_biomass_csv(df):
-                df = create_csv_from_df(df)
-                if df is None:
-                    return None
-                csv_data = df.to_csv(index=False)
-                return (csv_data, "biomass_table.csv")
+            def download_biomass_csv():
+                if hasattr(update_declining_increment, '_last_biomass_table'):
+                    df = update_declining_increment._last_biomass_table.copy()
+                    # Remove formatting for CSV export - this table has complex column names
+                    for col in df.columns:
+                        if df[col].dtype == 'object':
+                            try:
+                                # Try to convert formatted strings back to numbers
+                                df[col] = df[col].str.replace(',', '').astype(float)
+                            except:
+                                pass  # Keep as string if conversion fails
+                    
+                    temp_file = tempfile.NamedTemporaryFile(mode='w', suffix='.csv', delete=False)
+                    df.to_csv(temp_file.name, index=False)
+                    temp_file.close()
+                    return temp_file.name
+                return None
             
             download_results_btn.click(
                 download_results_csv,
-                inputs=[results_box],
-                outputs=[gr.File(label="Download Project Results", file_count="single", interactive=False)]
+                inputs=[],
+                outputs=[]
             )
             
             download_species_btn.click(
                 download_species_csv,
-                inputs=[species_box],
-                outputs=[gr.File(label="Download Species Results", file_count="single", interactive=False)]
+                inputs=[],
+                outputs=[]
             )
             
             download_survival_btn.click(
                 download_survival_csv,
-                inputs=[survival_box],
-                outputs=[gr.File(label="Download Surviving Trees", file_count="single", interactive=False)]
+                inputs=[],
+                outputs=[]
             )
             
             download_biomass_btn.click(
                 download_biomass_csv,
-                inputs=[biomass_debug_table],
-                outputs=[gr.File(label="Download Biomass Table", file_count="single", interactive=False)]
+                inputs=[],
+                outputs=[]
             )
             
             # Show initial results