James McCool commited on
Commit
9c68c67
·
1 Parent(s): ffc75f9

Refactor combo analysis controls in 'Manage Portfolio' to use a form for submission, improving user interaction and streamlining the analysis process.

Browse files
Files changed (1) hide show
  1. app.py +12 -10
app.py CHANGED
@@ -2008,16 +2008,18 @@ if selected_tab == 'Manage Portfolio':
2008
  st.subheader("Player Combinations")
2009
 
2010
  # Add controls for combo analysis
2011
- combo_size_col, columns_excluded_col, combo_analyze_col = st.columns(3)
2012
- with combo_size_col:
2013
- combo_size = st.selectbox("Combo Size", [2, 3], key='combo_size')
2014
- with columns_excluded_col:
2015
- excluded_cols_extended = st.multiselect("Exclude Columns?", st.session_state['display_frame'].drop(columns=excluded_cols).columns, key='excluded_cols_extended')
2016
- with combo_analyze_col:
2017
- if st.button("Analyze Combos", key='analyze_combos'):
2018
- st.session_state['combo_analysis'] = analyze_player_combos(
2019
- st.session_state['display_frame'], excluded_cols + excluded_cols_extended, combo_size
2020
- )
 
 
2021
 
2022
  # Display results
2023
  if 'combo_analysis' in st.session_state:
 
2008
  st.subheader("Player Combinations")
2009
 
2010
  # Add controls for combo analysis
2011
+ with st.form("combo_analysis_form"):
2012
+ combo_size_col, columns_excluded_col, combo_analyze_col = st.columns(3)
2013
+ with combo_size_col:
2014
+ combo_size = st.selectbox("Combo Size", [2, 3], key='combo_size')
2015
+ with columns_excluded_col:
2016
+ excluded_cols_extended = st.multiselect("Exclude Columns?", st.session_state['display_frame'].drop(columns=excluded_cols).columns, key='excluded_cols_extended')
2017
+ with combo_analyze_col:
2018
+ submitted = st.form_submit_button("Analyze Combos")
2019
+ if submitted:
2020
+ st.session_state['combo_analysis'] = analyze_player_combos(
2021
+ st.session_state['display_frame'], excluded_cols + excluded_cols_extended, combo_size
2022
+ )
2023
 
2024
  # Display results
2025
  if 'combo_analysis' in st.session_state: