Fix pandas warning on chained indexing

bbucior · bbucior · commit ed22a08f98c1 · 2021-01-19T20:21:09.000-05:00
diff --git a/src/server/pipeline/match_data.py b/src/server/pipeline/match_data.py
@@ -51,7 +51,7 @@ def start(connection, added_or_updated_rows):
                     "Source {} with ID {} is matching multiple groups in pdp_contacts ({})"
                     .format(row["source_type"], row["source_id"], str(row_matches["matching_id"].drop_duplicates()))
                 )
-        items_to_update["matching_id"][row_num] = row_group
+        items_to_update.loc[row_num, "matching_id"] = row_group
         # Updating local pdp_contacts dataframe instead of a roundtrip to postgres within the loop.
         # Indexing by iloc and vector of rows to keep the pd.DataFrame class and avoid implicit
         # casting to a single-typed pd.Series.

Original file line number	Diff line number	Diff line change
`@@ -51,7 +51,7 @@ def start(connection, added_or_updated_rows):`
`51`	`51`	`"Source {} with ID {} is matching multiple groups in pdp_contacts ({})"`
`52`	`52`	`.format(row["source_type"], row["source_id"], str(row_matches["matching_id"].drop_duplicates()))`
`53`	`53`	`)`
`54`		`- items_to_update["matching_id"][row_num] = row_group`
	`54`	`+ items_to_update.loc[row_num, "matching_id"] = row_group`
`55`	`55`	`# Updating local pdp_contacts dataframe instead of a roundtrip to postgres within the loop.`
`56`	`56`	`# Indexing by iloc and vector of rows to keep the pd.DataFrame class and avoid implicit`
`57`	`57`	`# casting to a single-typed pd.Series.`