Implement position limits and filtering logic in Handbuilder tab of app.py to manage player selection more effectively. This update enhances lineup management by preventing selection of players whose positions have reached their maximum count, improving user experience.

app.py

@@ -802,6 +802,29 @@ with tab3:
 with tab4:
     st.header("Handbuilder")
 
+    # --- LINEUP STATE ---
+    if 'handbuilder_lineup' not in st.session_state:
+        st.session_state['handbuilder_lineup'] = pd.DataFrame(columns=['Player', 'Position', 'Team', 'Salary', 'Median', 'Own%'])
+    if 'handbuilder_select_key' not in st.session_state:
+        st.session_state['handbuilder_select_key'] = 0
+
+    # --- POSITION LIMITS ---
+    position_limits = {
+        'SP': 2,
+        # Add other positions and their limits as needed
+        # e.g. 'C': 1, '1B': 1, '2B': 1, '3B': 1, 'SS': 1, 'OF': 3
+    }
+
+    # Count positions in the current lineup
+    lineup = st.session_state['handbuilder_lineup']
+    position_counts = lineup['Position'].value_counts() if not lineup.empty else {}
+
+    # Build a set of positions that have reached their max
+    positions_at_max = set()
+    for pos, max_count in position_limits.items():
+        if position_counts.get(pos, 0) >= max_count:
+            positions_at_max.add(pos)
+
     # --- TEAM FILTER UI ---
     all_teams = sorted(dk_roo['Team'].unique())
     st.markdown("**Toggle teams to include:**")
@@ -828,29 +851,6 @@ with tab4:
     if positions_at_max:
         player_select_df = player_select_df[~player_select_df['Position'].isin(positions_at_max)]
 
-    # --- LINEUP STATE ---
-    if 'handbuilder_lineup' not in st.session_state:
-        st.session_state['handbuilder_lineup'] = pd.DataFrame(columns=['Player', 'Position', 'Team', 'Salary', 'Median', 'Own%'])
-    if 'handbuilder_select_key' not in st.session_state:
-        st.session_state['handbuilder_select_key'] = 0
-
-    # --- POSITION LIMITS ---
-    position_limits = {
-        'SP': 2,
-        # Add other positions and their limits as needed
-        # e.g. 'C': 1, '1B': 1, '2B': 1, '3B': 1, 'SS': 1, 'OF': 3
-    }
-
-    # Count positions in the current lineup
-    lineup = st.session_state['handbuilder_lineup']
-    position_counts = lineup['Position'].value_counts() if not lineup.empty else {}
-
-    # Build a set of positions that have reached their max
-    positions_at_max = set()
-    for pos, max_count in position_limits.items():
-        if position_counts.get(pos, 0) >= max_count:
-            positions_at_max.add(pos)
-
     col1, col2 = st.columns([1, 2])
     with col2:
         st.subheader("Player Select")