fix to pre-commit

Author: aeturrell

geo-intro.ipynb

@@ -942,26 +942,17 @@
    "metadata": {},
    "outputs": [],
    "source": [
-    "geolocator = Nominatim(user_agent=\"coding-for-economists/1.0\")\n",
-    "\n",
-    "# Function to safely geocode a place name\n",
-    "# We use a local loop here because the Photon API (photon.komoot.io) frequently times out.\n",
-    "# Using Nominatim directly gives us more control over timeouts and error handling.\n",
-    "def get_geo(point_name):\n",
-    "    try:\n",
-    "        loc = geolocator.geocode(f\"{point_name}, NY\", timeout=10)\n",
-    "        return Point(loc.latitude, loc.longitude) if loc else None\n",
-    "    except Exception:\n",
-    "        return None\n",
-    "\n",
-    "boros_df['geometry'] = boros_df.boro_name.apply(get_geo)\n",
-    "# Drop any failed geocode attempts before converting to GeoDataFrame\n",
-    "boroughs_gdf = gpd.GeoDataFrame(\n",
-    "    boros_df.dropna(subset=['geometry']), \n",
-    "    geometry='geometry', \n",
-    "    crs='EPSG:4326'\n",
-    ")\n",
-    "boroughs_gdf\n"
+    "# If you don't have it, install geopy using 'pip install geopy'\n",
+    "from geopy.geocoders import Nominatim\n",
+    "\n",
+    "# Create a geolocator using Open Street Map (aka Nominatim)\n",
+    "# Use your own user agent identifier here\n",
+    "geolocator = Nominatim(user_agent=\"codeforecon\")\n",
+    "\n",
+    "# Pass an address to retrieve full location information:\n",
+    "location = geolocator.geocode(\"Bank of England\")\n",
+    "\n",
+    "location.address"
    ]
   },
   {
@@ -1034,26 +1025,14 @@
    "metadata": {},
    "outputs": [],
    "source": [
-    "geolocator = Nominatim(user_agent=\"coding-for-economists/1.0\")\n",
-    "\n",
-    "# Function to safely geocode a place name\n",
-    "# We use a local loop here because the Photon API (photon.komoot.io) frequently times out.\n",
-    "# Using Nominatim directly gives us more control over timeouts and error handling.\n",
-    "def get_geo(point_name):\n",
-    "    try:\n",
-    "        loc = geolocator.geocode(f\"{point_name}, NY\", timeout=10)\n",
-    "        return Point(loc.latitude, loc.longitude) if loc else None\n",
-    "    except Exception:\n",
-    "        return None\n",
-    "\n",
-    "boros_df['geometry'] = boros_df.boro_name.apply(get_geo)\n",
-    "# Drop any failed geocode attempts before converting to GeoDataFrame\n",
-    "boroughs_gdf = gpd.GeoDataFrame(\n",
-    "    boros_df.dropna(subset=['geometry']), \n",
-    "    geometry='geometry', \n",
-    "    crs='EPSG:4326'\n",
-    ")\n",
-    "boroughs_gdf\n"
+    "# Nominatim (OpenStreetMap) is the provider here; it requires a user agent\n",
+    "boros_df[[\"geometry\", \"address\"]] = gpd.tools.geocode(\n",
+    "    boros_df.boro_name + \", New York\",\n",
+    "    provider=\"nominatim\",\n",
+    "    user_agent=\"coding-for-economists/1.0\",\n",
+    "    timeout=10,\n",
+    ").set_geometry(\"geometry\")\n",
+    "boros_df"
    ]
   },
   {