contrib/sync-places: filter tags during creation

In older versions of python, when iterating over a dictionary, a copy
would be created allowing us to modify the dictionary while we're
iterating over it. Newer versions however do not make this copy and now
raise runtime errors when the keys change while we're iterating over
them.

Transform the key/values during creation of the 'tags' dictionary to
avoid having to delete keys afterwards.

Signed-off-by: Bryan Brattlof <[email protected]>

Author: bryanbrattlof

contrib/sync-places.py

@@ -117,11 +117,7 @@ async def do_sync(session, args):
                         await session.sync_with_coordinator()
                     changed = True
 
-            tags = config["places"][name].get("tags", {}).copy()
-            for k, v in tags.items():
-                if not isinstance(k, str) or not isinstance(v, str):
-                    del(tags[k])
-                    tags[str(k)] = str(v)
+            tags = { str(k): str(v) for k, v in config["places"][name].get("tags", {}).items() }
 
             if place_tags != tags:
                 print(