James McCool commited on
Commit
d2ba2bf
·
1 Parent(s): 1817a5f

Refactor player and stack frame calculations in app.py

Browse files

- Updated the logic for calculating player and stack frame displays to use DataFrame indexing, improving accuracy in the 'Relation to the field' view.
- Implemented subtraction with fill values to handle missing data more effectively, enhancing data integrity in the application.
- Ensured consistent resetting of indices for cleaner DataFrame presentation, improving overall user experience.

Files changed (1) hide show
  1. app.py +16 -4
app.py CHANGED
@@ -247,7 +247,10 @@ with tab2:
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_display = st.session_state['player_frame'] - st.session_state['field_player_frame']
 
 
 
251
  st.dataframe(player_frame_display.
252
  sort_values(by='Exposure Overall', ascending=False).
253
  style.background_gradient(cmap='RdYlGn').
@@ -258,7 +261,10 @@ with tab2:
258
  if player_view_var == 'Percentage used':
259
  player_frame_display = st.session_state['player_frame']
260
  elif player_view_var == 'Relation to the field':
261
- player_frame_display = st.session_state['player_frame'] - st.session_state['field_player_frame']
 
 
 
262
  st.dataframe(player_frame_display.
263
  sort_values(by='Exposure Overall', ascending=False).
264
  style.background_gradient(cmap='RdYlGn').
@@ -272,7 +278,10 @@ with tab2:
272
  if stack_view_var == 'Percentage used':
273
  stack_frame_display = st.session_state['stack_frame']
274
  elif stack_view_var == 'Relation to the field':
275
- stack_frame_display = st.session_state['stack_frame'] - st.session_state['field_stack_frame']
 
 
 
276
  st.dataframe(stack_frame_display.
277
  sort_values(by='Exposure Overall', ascending=False).
278
  style.background_gradient(cmap='RdYlGn').
@@ -283,7 +292,10 @@ with tab2:
283
  if stack_view_var == 'Percentage used':
284
  stack_frame_display = st.session_state['stack_frame']
285
  elif stack_view_var == 'Relation to the field':
286
- stack_frame_display = st.session_state['stack_frame'] - st.session_state['field_stack_frame']
 
 
 
287
  st.dataframe(stack_frame_display.
288
  sort_values(by='Exposure Overall', ascending=False).
289
  style.background_gradient(cmap='RdYlGn').
 
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(drop=True)
254
  st.dataframe(player_frame_display.
255
  sort_values(by='Exposure Overall', ascending=False).
256
  style.background_gradient(cmap='RdYlGn').
 
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(drop=True)
268
  st.dataframe(player_frame_display.
269
  sort_values(by='Exposure Overall', ascending=False).
270
  style.background_gradient(cmap='RdYlGn').
 
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').
 
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').