James McCool commited on
Commit
447182c
·
1 Parent(s): a134635

Refactor ownership and points dictionary updates in find_name_mismatches function

Browse files

- Consolidated the logic for creating ownership and fpts dictionaries to ensure accurate mapping of player names to their respective ownership and points.
- Removed redundant variable assignments to streamline the code and improve readability, enhancing overall data integrity during name matching operations.

global_func/find_name_mismatches.py CHANGED
@@ -83,10 +83,9 @@ def find_name_mismatches(contest_df, projections_df, ownership_df, fpts_df):
83
 
84
  if contest_name in ownership_raw:
85
  ownership_raw['Player'] = ownership_raw['Player'].replace(contest_name, projection_name)
86
- ownership_dict = dict(zip(ownership_raw['Player'], ownership_raw['Own']))
87
  if contest_name in fpts_raw:
88
  fpts_raw['Player'] = fpts_raw['Player'].replace(contest_name, projection_name)
89
- fpts_dict = dict(zip(fpts_raw['Player'], fpts_raw['FPTS']))
90
 
91
  # Process manual selections
92
  for projection_name, selection in selections.items():
@@ -97,17 +96,20 @@ def find_name_mismatches(contest_df, projections_df, ownership_df, fpts_df):
97
 
98
  if contest_name in ownership_raw:
99
  ownership_raw['Player'] = ownership_raw['Player'].replace(contest_name, projection_name)
100
- ownership_dict = dict(zip(ownership_raw['Player'], ownership_raw['Own']))
101
  if contest_name in fpts_raw:
102
  fpts_raw['Player'] = fpts_raw['Player'].replace(contest_name, projection_name)
103
- fpts_dict = dict(zip(fpts_raw['Player'], fpts_raw['FPTS']))
104
 
105
  st.success(f"Replaced '{selected_name}' with '{projection_name}'")
106
 
 
 
 
107
  st.success("All changes applied successfully!")
108
  return contest_raw, projections_raw, ownership_dict, fpts_dict
109
 
110
  # Return the current state if form hasn't been submitted yet
 
 
111
  return contest_raw, projections_raw, ownership_dict, fpts_dict
112
  else:
113
  st.success("All players have been automatically matched!")
@@ -118,9 +120,10 @@ def find_name_mismatches(contest_df, projections_df, ownership_df, fpts_df):
118
 
119
  if contest_name in ownership_raw:
120
  ownership_raw['Player'] = ownership_raw['Player'].replace(contest_name, projection_name)
121
- ownership_dict = dict(zip(ownership_raw['Player'], ownership_raw['Own']))
122
  if contest_name in fpts_raw:
123
  fpts_raw['Player'] = fpts_raw['Player'].replace(contest_name, projection_name)
124
- fpts_dict = dict(zip(fpts_raw['Player'], fpts_raw['FPTS']))
 
 
125
 
126
  return contest_raw, projections_raw, ownership_dict, fpts_dict
 
83
 
84
  if contest_name in ownership_raw:
85
  ownership_raw['Player'] = ownership_raw['Player'].replace(contest_name, projection_name)
86
+
87
  if contest_name in fpts_raw:
88
  fpts_raw['Player'] = fpts_raw['Player'].replace(contest_name, projection_name)
 
89
 
90
  # Process manual selections
91
  for projection_name, selection in selections.items():
 
96
 
97
  if contest_name in ownership_raw:
98
  ownership_raw['Player'] = ownership_raw['Player'].replace(contest_name, projection_name)
 
99
  if contest_name in fpts_raw:
100
  fpts_raw['Player'] = fpts_raw['Player'].replace(contest_name, projection_name)
 
101
 
102
  st.success(f"Replaced '{selected_name}' with '{projection_name}'")
103
 
104
+ ownership_dict = dict(zip(ownership_raw['Player'], ownership_raw['Own']))
105
+ fpts_dict = dict(zip(fpts_raw['Player'], fpts_raw['FPTS']))
106
+
107
  st.success("All changes applied successfully!")
108
  return contest_raw, projections_raw, ownership_dict, fpts_dict
109
 
110
  # Return the current state if form hasn't been submitted yet
111
+ ownership_dict = dict(zip(ownership_raw['Player'], ownership_raw['Own']))
112
+ fpts_dict = dict(zip(fpts_raw['Player'], fpts_raw['FPTS']))
113
  return contest_raw, projections_raw, ownership_dict, fpts_dict
114
  else:
115
  st.success("All players have been automatically matched!")
 
120
 
121
  if contest_name in ownership_raw:
122
  ownership_raw['Player'] = ownership_raw['Player'].replace(contest_name, projection_name)
 
123
  if contest_name in fpts_raw:
124
  fpts_raw['Player'] = fpts_raw['Player'].replace(contest_name, projection_name)
125
+
126
+ ownership_dict = dict(zip(ownership_raw['Player'], ownership_raw['Own']))
127
+ fpts_dict = dict(zip(fpts_raw['Player'], fpts_raw['FPTS']))
128
 
129
  return contest_raw, projections_raw, ownership_dict, fpts_dict