James McCool commited on
Commit
371a9aa
·
1 Parent(s): 302d0d9

Update trimming dictionary initialization in app.py: ensure consistent handling of maximum values for ownership types during portfolio updates, improving state management and preventing data conflicts.

Browse files
Files changed (1) hide show
  1. app.py +16 -10
app.py CHANGED
@@ -944,6 +944,14 @@ with tab2:
944
  if submitted:
945
  st.write('initiated')
946
  st.session_state['portfolio'] = predict_dupes(st.session_state['portfolio'], map_dict, site_var, type_var, Contest_Size, strength_var, sport_var)
 
 
 
 
 
 
 
 
947
  st.session_state['portfolio'] = st.session_state['portfolio'][st.session_state['portfolio']['Dupes'] <= max_dupes]
948
  st.session_state['portfolio'] = st.session_state['portfolio'][st.session_state['portfolio']['salary'] >= min_salary]
949
  st.session_state['portfolio'] = st.session_state['portfolio'][st.session_state['portfolio']['salary'] <= max_salary]
@@ -979,16 +987,14 @@ with tab2:
979
  with col2:
980
  st.write('initiated')
981
  st.session_state['portfolio'] = predict_dupes(st.session_state['portfolio'], map_dict, site_var, type_var, Contest_Size, strength_var, sport_var)
982
- if 'trimming_dict_maxes' in st.session_state:
983
- del st.session_state['trimming_dict_maxes']
984
-
985
- st.session_state['trimming_dict_maxes'] = {
986
- 'Own': st.session_state['portfolio']['Own'].max(),
987
- 'Geomean': st.session_state['portfolio']['Geomean'].max(),
988
- 'Weighted Own': st.session_state['portfolio']['Weighted Own'].max(),
989
- 'median': st.session_state['portfolio']['median'].max(),
990
- 'Finish_percentile': st.session_state['portfolio']['Finish_percentile'].max()
991
- }
992
  st.session_state['portfolio'] = st.session_state['portfolio'][st.session_state['portfolio']['Dupes'] <= max_dupes]
993
  st.session_state['portfolio'] = st.session_state['portfolio'][st.session_state['portfolio']['salary'] >= min_salary]
994
  st.session_state['portfolio'] = st.session_state['portfolio'][st.session_state['portfolio']['salary'] <= max_salary]
 
944
  if submitted:
945
  st.write('initiated')
946
  st.session_state['portfolio'] = predict_dupes(st.session_state['portfolio'], map_dict, site_var, type_var, Contest_Size, strength_var, sport_var)
947
+ if 'trimming_dict_maxes' not in st.session_state:
948
+ st.session_state['trimming_dict_maxes'] = {
949
+ 'Own': st.session_state['portfolio']['Own'].max(),
950
+ 'Geomean': st.session_state['portfolio']['Geomean'].max(),
951
+ 'Weighted Own': st.session_state['portfolio']['Weighted Own'].max(),
952
+ 'median': st.session_state['portfolio']['median'].max(),
953
+ 'Finish_percentile': st.session_state['portfolio']['Finish_percentile'].max()
954
+ }
955
  st.session_state['portfolio'] = st.session_state['portfolio'][st.session_state['portfolio']['Dupes'] <= max_dupes]
956
  st.session_state['portfolio'] = st.session_state['portfolio'][st.session_state['portfolio']['salary'] >= min_salary]
957
  st.session_state['portfolio'] = st.session_state['portfolio'][st.session_state['portfolio']['salary'] <= max_salary]
 
987
  with col2:
988
  st.write('initiated')
989
  st.session_state['portfolio'] = predict_dupes(st.session_state['portfolio'], map_dict, site_var, type_var, Contest_Size, strength_var, sport_var)
990
+ if 'trimming_dict_maxes' not in st.session_state:
991
+ st.session_state['trimming_dict_maxes'] = {
992
+ 'Own': st.session_state['portfolio']['Own'].max(),
993
+ 'Geomean': st.session_state['portfolio']['Geomean'].max(),
994
+ 'Weighted Own': st.session_state['portfolio']['Weighted Own'].max(),
995
+ 'median': st.session_state['portfolio']['median'].max(),
996
+ 'Finish_percentile': st.session_state['portfolio']['Finish_percentile'].max()
997
+ }
 
 
998
  st.session_state['portfolio'] = st.session_state['portfolio'][st.session_state['portfolio']['Dupes'] <= max_dupes]
999
  st.session_state['portfolio'] = st.session_state['portfolio'][st.session_state['portfolio']['salary'] >= min_salary]
1000
  st.session_state['portfolio'] = st.session_state['portfolio'][st.session_state['portfolio']['salary'] <= max_salary]