Update src/streamlit_app.py

src/streamlit_app.py

@@ -9,11 +9,9 @@ st.set_page_config(layout="wide", page_title="IMDb Cinematic Trends")
 # Enable Altair Dark Theme for visibility on black backgrounds
 alt.themes.enable('dark')
 
-# Data Loading (Cached for Speed)
+# Data Loading 
 @st.cache_data
 def load_all_data():
-    # Using relative paths
-    # If files are in a 'src' folder, we check there first
     main_path = 'src/merged_imdb_sample.csv' if os.path.exists('src/merged_imdb_sample.csv') else 'merged_imdb_sample.csv'
     ctx_path = 'src/principals_context_sample.csv' if os.path.exists('src/principals_context_sample.csv') else 'principals_context_sample.csv'
     
@@ -75,8 +73,6 @@ genres and decades, along with the top directors for them, and how those ratings
 st.header("1. Interactive Trend Explorer")
 st.info("💡 **Interaction:** Click a cell in the Heatmap to filter the Top Directors chart below.")
 
-# 1. Define selection WITHOUT a default value (prevents the Schema Error)
-# 'toggle=False' makes it so clicking outside doesn't clear the selection
 selection = alt.selection_point(
     fields=['genre_list', 'decade'], 
     name='cell',
@@ -99,15 +95,17 @@ heatmap = alt.Chart(df).mark_rect().encode(
     title="Average Movie Ratings by Genre and Decade"
 ).add_params(selection)
 
-# 3. Bar Chart with "Fallback" Logic
+
+# 3. Bar Chart
 bar_chart = alt.Chart(df).mark_bar().encode(
+    # Use the aggregated field name here
     x=alt.X('total_votes:Q', title='Total Popularity (Votes)'),
     y=alt.Y('directorName:N', sort='-x', title='Director', axis=alt.Axis(labelLimit=200)),
     color=alt.value('#4682B4'),
-    tooltip=['directorName', 'sum(numVotes)']
-).transform_filter(
-    # If nothing is selected, this filter now returns TRUE for everything 
-    selection
+    # Changed 'sum(numVotes)' to 'total_votes'
+    tooltip=['directorName', alt.Tooltip('total_votes:Q', format=',')] 
+).transform_filter( 
+    selection # Ensure 'selection' is defined above this block!
 ).transform_aggregate(
     total_votes='sum(numVotes)',
     groupby=['directorName']
@@ -148,6 +146,7 @@ It suggests that movie records are not only about the films themselves, but also
 contributors, such as actors, actresses, directors, and writers, are presented to the public.
 """)
 
+
 # Contextual Charts
 st.header("2. Production Context")
 ctx_col1, ctx_col2 = st.columns(2)
@@ -178,6 +177,8 @@ with ctx_col2:
     st.altair_chart(collab_density, use_container_width=True)
     st.caption("Source: IMDb Principals Sample. Most successful films require 10-25 key leaders.")
 
+
+
 # Narrative Paragraph
 st.markdown("""
 ### The Final Story: Complexity and Quality