James McCool commited on
Commit
ecb542f
·
1 Parent(s): f575676

Refactor player selection logic in app.py

Browse files

- Changed the method of selecting top players from using `nlargest` to `head`, improving performance and clarity in the code.
- This adjustment ensures that the top players are selected based on their ranking in the dataframe, enhancing data processing efficiency.

Files changed (1) hide show
  1. app.py +4 -4
app.py CHANGED
@@ -228,10 +228,10 @@ with tab2:
228
 
229
  for col in player_columns:
230
  contest_players = working_df.copy()
231
- players_1per = working_df.nlargest(n=int(len(working_df) * 0.01), columns='actual_fpts')
232
- players_5per = working_df.nlargest(n=int(len(working_df) * 0.05), columns='actual_fpts')
233
- players_10per = working_df.nlargest(n=int(len(working_df) * 0.10), columns='actual_fpts')
234
- players_20per = working_df.nlargest(n=int(len(working_df) * 0.20), columns='actual_fpts')
235
  player_counts = pd.Series(list(contest_players[player_columns].values.flatten())).value_counts()
236
  player_1per_counts = pd.Series(list(players_1per[player_columns].values.flatten())).value_counts()
237
  player_5per_counts = pd.Series(list(players_5per[player_columns].values.flatten())).value_counts()
 
228
 
229
  for col in player_columns:
230
  contest_players = working_df.copy()
231
+ players_1per = working_df.head(int(len(working_df) * 0.01))
232
+ players_5per = working_df.head(int(len(working_df) * 0.05))
233
+ players_10per = working_df.head(int(len(working_df) * 0.10))
234
+ players_20per = working_df.head(int(len(working_df) * 0.20))
235
  player_counts = pd.Series(list(contest_players[player_columns].values.flatten())).value_counts()
236
  player_1per_counts = pd.Series(list(players_1per[player_columns].values.flatten())).value_counts()
237
  player_5per_counts = pd.Series(list(players_5per[player_columns].values.flatten())).value_counts()