James McCool commited on
Commit
d4f6599
·
1 Parent(s): d5219a8

Refactor session state management by removing unnecessary copy operations for origin and working portfolios.

Browse files
Files changed (1) hide show
  1. app.py +2 -2
app.py CHANGED
@@ -567,7 +567,7 @@ if selected_tab == 'Data Load':
567
  'cpt_proj_map':dict(zip(st.session_state['projections_df']['player_names'], st.session_state['projections_df']['median'] * 1.5)),
568
  'cpt_own_map':dict(zip(st.session_state['projections_df']['player_names'], st.session_state['projections_df']['captain ownership']))
569
  }
570
- st.session_state['origin_portfolio'] = st.session_state['portfolio'].copy()
571
  buffer = io.BytesIO()
572
  st.session_state['portfolio'].to_parquet(buffer, compression='snappy')
573
  st.session_state['origin_portfolio'] = buffer.getvalue()
@@ -1092,7 +1092,7 @@ if selected_tab == 'Manage Portfolio':
1092
  with reset_port_col:
1093
  if st.button('Reset Portfolio', key='reset_port'):
1094
  st.session_state['settings_base'] = True
1095
- st.session_state['working_frame'] = st.session_state['base_frame'].copy()
1096
  with recalc_div_col:
1097
  if st.button("Recalculate Diversity"):
1098
  st.session_state['display_frame']['Diversity'] = recalc_diversity(st.session_state['display_frame'], st.session_state['player_columns'])
 
567
  'cpt_proj_map':dict(zip(st.session_state['projections_df']['player_names'], st.session_state['projections_df']['median'] * 1.5)),
568
  'cpt_own_map':dict(zip(st.session_state['projections_df']['player_names'], st.session_state['projections_df']['captain ownership']))
569
  }
570
+ st.session_state['origin_portfolio'] = st.session_state['portfolio']
571
  buffer = io.BytesIO()
572
  st.session_state['portfolio'].to_parquet(buffer, compression='snappy')
573
  st.session_state['origin_portfolio'] = buffer.getvalue()
 
1092
  with reset_port_col:
1093
  if st.button('Reset Portfolio', key='reset_port'):
1094
  st.session_state['settings_base'] = True
1095
+ st.session_state['working_frame'] = st.session_state['base_frame']
1096
  with recalc_div_col:
1097
  if st.button("Recalculate Diversity"):
1098
  st.session_state['display_frame']['Diversity'] = recalc_diversity(st.session_state['display_frame'], st.session_state['player_columns'])