James McCool commited on
Commit
6d2f6bb
·
1 Parent(s): 3e312de

Refactor player and stack frame display logic in app.py

Browse files

- Removed redundant view selection logic for player and stack frames, simplifying the code and enhancing readability.
- Streamlined the display of dataframes by directly using session state variables, improving performance and clarity in the output.
- Maintained the overall functionality while enhancing user experience through cleaner data presentation.

Files changed (1) hide show
  1. app.py +8 -38
app.py CHANGED
@@ -240,66 +240,36 @@ with tab2:
240
  with st.container():
241
  tab1, tab2, tab3 = st.tabs(['Player Used Info', 'Stack Used Info', 'Duplication Info'])
242
  with tab1:
243
- player_view_var = st.radio("View Exposures by:", ['Percentage used', 'Relation to the field'], key='player_view_var')
244
 
245
  if entry_parse_var == 'All':
246
  st.session_state['player_frame'] = create_player_exposures(working_df, player_columns)
247
- if player_view_var == 'Percentage used':
248
- player_frame_display = st.session_state['player_frame']
249
- elif player_view_var == 'Relation to the field':
250
- player_frame = st.session_state['player_frame'].set_index('Player')
251
- field_player_frame = st.session_state['field_player_frame'].set_index('Player')
252
- player_frame_display = player_frame.subtract(field_player_frame, fill_value=0)
253
- player_frame_display = player_frame_display.reset_index()
254
- st.dataframe(player_frame_display.
255
  sort_values(by='Exposure Overall', ascending=False).
256
  style.background_gradient(cmap='RdYlGn').
257
- format(formatter='{:.2%}', subset=player_frame_display.select_dtypes(include=['number']).columns),
258
  hide_index=True)
259
  else:
260
  st.session_state['player_frame'] = create_player_exposures(working_df, player_columns, entry_names)
261
- if player_view_var == 'Percentage used':
262
- player_frame_display = st.session_state['player_frame']
263
- elif player_view_var == 'Relation to the field':
264
- player_frame = st.session_state['player_frame'].set_index('Player')
265
- field_player_frame = st.session_state['field_player_frame'].set_index('Player')
266
- player_frame_display = player_frame.subtract(field_player_frame, fill_value=0)
267
- player_frame_display = player_frame_display.reset_index()
268
- st.dataframe(player_frame_display.
269
  sort_values(by='Exposure Overall', ascending=False).
270
  style.background_gradient(cmap='RdYlGn').
271
- format(formatter='{:.2%}', subset=player_frame_display.select_dtypes(include=['number']).columns),
272
  hide_index=True)
273
  with tab2:
274
- stack_view_var = st.radio('View Stack Exposures by:', ['Percentage used', 'Relation to the field'], key = 'stack_view_var')
275
 
276
  if entry_parse_var == 'All':
277
  st.session_state['stack_frame'] = create_stack_exposures(working_df)
278
- if stack_view_var == 'Percentage used':
279
- stack_frame_display = st.session_state['stack_frame']
280
- elif stack_view_var == 'Relation to the field':
281
- stack_frame = st.session_state['stack_frame'].set_index('Stack')
282
- field_stack_frame = st.session_state['field_stack_frame'].set_index('Stack')
283
- stack_frame_display = stack_frame.subtract(field_stack_frame, fill_value=0)
284
- stack_frame_display = stack_frame_display.reset_index(drop=True)
285
- st.dataframe(stack_frame_display.
286
  sort_values(by='Exposure Overall', ascending=False).
287
  style.background_gradient(cmap='RdYlGn').
288
- format(formatter='{:.2%}', subset=stack_frame_display.select_dtypes(include=['number']).columns),
289
  hide_index=True)
290
  else:
291
  st.session_state['stack_frame'] = create_stack_exposures(working_df, entry_names)
292
- if stack_view_var == 'Percentage used':
293
- stack_frame_display = st.session_state['stack_frame']
294
- elif stack_view_var == 'Relation to the field':
295
- stack_frame = st.session_state['stack_frame'].set_index('Stack')
296
- field_stack_frame = st.session_state['field_stack_frame'].set_index('Stack')
297
- stack_frame_display = stack_frame.subtract(field_stack_frame, fill_value=0)
298
- stack_frame_display = stack_frame_display.reset_index(drop=True)
299
- st.dataframe(stack_frame_display.
300
  sort_values(by='Exposure Overall', ascending=False).
301
  style.background_gradient(cmap='RdYlGn').
302
- format(formatter='{:.2%}', subset=stack_frame_display.select_dtypes(include=['number']).columns),
303
  hide_index=True)
304
  with tab3:
305
  st.write('holding')
 
240
  with st.container():
241
  tab1, tab2, tab3 = st.tabs(['Player Used Info', 'Stack Used Info', 'Duplication Info'])
242
  with tab1:
 
243
 
244
  if entry_parse_var == 'All':
245
  st.session_state['player_frame'] = create_player_exposures(working_df, player_columns)
246
+ st.dataframe(st.session_state['player_frame'].
 
 
 
 
 
 
 
247
  sort_values(by='Exposure Overall', ascending=False).
248
  style.background_gradient(cmap='RdYlGn').
249
+ format(formatter='{:.2%}', subset=st.session_state['player_frame'].select_dtypes(include=['number']).columns),
250
  hide_index=True)
251
  else:
252
  st.session_state['player_frame'] = create_player_exposures(working_df, player_columns, entry_names)
253
+ st.dataframe(st.session_state['player_frame'].
 
 
 
 
 
 
 
254
  sort_values(by='Exposure Overall', ascending=False).
255
  style.background_gradient(cmap='RdYlGn').
256
+ format(formatter='{:.2%}', subset=st.session_state['player_frame'].select_dtypes(include=['number']).columns),
257
  hide_index=True)
258
  with tab2:
 
259
 
260
  if entry_parse_var == 'All':
261
  st.session_state['stack_frame'] = create_stack_exposures(working_df)
262
+ st.dataframe(st.session_state['stack_frame'].
 
 
 
 
 
 
 
263
  sort_values(by='Exposure Overall', ascending=False).
264
  style.background_gradient(cmap='RdYlGn').
265
+ format(formatter='{:.2%}', subset=st.session_state['stack_frame'].select_dtypes(include=['number']).columns),
266
  hide_index=True)
267
  else:
268
  st.session_state['stack_frame'] = create_stack_exposures(working_df, entry_names)
269
+ st.dataframe(st.session_state['stack_frame'].
 
 
 
 
 
 
 
270
  sort_values(by='Exposure Overall', ascending=False).
271
  style.background_gradient(cmap='RdYlGn').
272
+ format(formatter='{:.2%}', subset=st.session_state['stack_frame'].select_dtypes(include=['number']).columns),
273
  hide_index=True)
274
  with tab3:
275
  st.write('holding')