James McCool commited on
Commit
2ba2e65
·
1 Parent(s): 30e81ef

Fix session state assignment in Manage Portfolio tab to correctly reference base_frame_names for working_frame and new_base_name.

Browse files
Files changed (1) hide show
  1. app.py +2 -2
app.py CHANGED
@@ -1750,7 +1750,7 @@ if selected_tab == 'Manage Portfolio':
1750
  with st.popover("Reset Portfolio"):
1751
  st.markdown("choose a base to reset to:")
1752
  base_choice = st.selectbox("Base Choice", options=st.session_state['base_frame_names'], index=0)
1753
- st.session_state['working_frame'] = st.session_state[f'{base_choice}']
1754
  with recalc_div_col:
1755
  if st.button("Recalculate Diversity"):
1756
  st.session_state['display_frame']['Diversity'] = recalc_diversity(st.session_state['display_frame'], st.session_state['player_columns'])
@@ -1761,7 +1761,7 @@ if selected_tab == 'Manage Portfolio':
1761
  if f'{new_base_name}' not in st.session_state['base_frame_names']:
1762
  st.session_state['base_frame_names'].append(f'{new_base_name}')
1763
  st.session_state['base_frame_names'] = sorted(st.session_state['base_frame_names'])
1764
- st.session_state[f'{new_base_name}'] = st.session_state['working_frame']
1765
  else:
1766
  st.error("Base name already exists")
1767
 
 
1750
  with st.popover("Reset Portfolio"):
1751
  st.markdown("choose a base to reset to:")
1752
  base_choice = st.selectbox("Base Choice", options=st.session_state['base_frame_names'], index=0)
1753
+ st.session_state['working_frame'] = st.session_state['base_frame_names'][base_choice]
1754
  with recalc_div_col:
1755
  if st.button("Recalculate Diversity"):
1756
  st.session_state['display_frame']['Diversity'] = recalc_diversity(st.session_state['display_frame'], st.session_state['player_columns'])
 
1761
  if f'{new_base_name}' not in st.session_state['base_frame_names']:
1762
  st.session_state['base_frame_names'].append(f'{new_base_name}')
1763
  st.session_state['base_frame_names'] = sorted(st.session_state['base_frame_names'])
1764
+ st.session_state['base_frame_names'][new_base_name] = st.session_state['working_frame']
1765
  else:
1766
  st.error("Base name already exists")
1767