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.
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 |
-
|
| 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=
|
| 258 |
hide_index=True)
|
| 259 |
else:
|
| 260 |
st.session_state['player_frame'] = create_player_exposures(working_df, player_columns, entry_names)
|
| 261 |
-
|
| 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=
|
| 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 |
-
|
| 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=
|
| 289 |
hide_index=True)
|
| 290 |
else:
|
| 291 |
st.session_state['stack_frame'] = create_stack_exposures(working_df, entry_names)
|
| 292 |
-
|
| 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=
|
| 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')
|