builds broken due to stopped service; now fixed

Author: aeturrell

geo-intro.ipynb

@@ -942,17 +942,26 @@
    "metadata": {},
    "outputs": [],
    "source": [
-    "# 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"
+    "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"
    ]
   },
   {
@@ -1025,11 +1034,26 @@
    "metadata": {},
    "outputs": [],
    "source": [
-    "boros_df[[\"geometry\", \"address\"]] = gpd.tools.geocode(\n",
-    "    boros_df.boro_name,\n",
-    "    provider=\"photon\",\n",
-    ").set_geometry(\"geometry\")\n",
-    "boros_df"
+    "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"
    ]
   },
   {