add numpydoc validation to pre-commit

.pre-commit-config.yaml

@@ -29,6 +29,11 @@ repos:
       - id: yamllint
         args: [--strict, --config-file=./tests/.yamllint.yml]
 
+  - repo: https://github.com/numpy/numpydoc
+    rev: v1.8.0
+    hooks:
+      - id: numpydoc-validation
+
   - repo: https://github.com/astral-sh/ruff-pre-commit
     rev: v0.9.1
     hooks:

environments/make-env-files.py

@@ -38,7 +38,7 @@ def make_requirement(
     Parameters
     ----------
     requirement
-        A requirement object
+        A requirement object.
     pin_exact
         If True, pin requirement to version rather than using existing
         specifier. Allows you to convert minimum versions to pinned versions.
@@ -49,6 +49,7 @@ def make_requirement(
     Returns
     -------
     requirement_str
+        The requirement's name and its specifier(s).
     """
     specifiers = list(requirement.specifier)
     if pin_exact and len(specifiers) == 1:

osmnx/convert.py

@@ -223,7 +223,6 @@ def _validate_node_edge_gdfs(
         GeoDataFrame of graph nodes uniquely indexed by `osmid`.
     gdf_edges
         GeoDataFrame of graph edges uniquely multi-indexed by `(u, v, key)`.
-    graph_attrs
 
     Returns
     -------

osmnx/features.py

@@ -351,10 +351,10 @@ def features_from_xml(
     ----------
     filepath
         Path to file containing OSM XML data.
-    tags
-        Query tags to optionally filter the final GeoDataFrame.
     polygon
         Spatial boundaries to optionally filter the final GeoDataFrame.
+    tags
+        Query tags to optionally filter the final GeoDataFrame.
     encoding
         The OSM XML file's character encoding.
 

osmnx/graph.py

@@ -599,9 +599,6 @@ def _create_graph(
     ----------
     response_jsons
         Iterable of JSON responses from the Overpass API.
-    retain_all
-        If True, return the entire graph even if it is not connected.
-        Otherwise, retain only the largest weakly connected component.
     bidirectional
         If True, create bidirectional edges for one-way streets.
 

osmnx/plot.py

@@ -325,7 +325,7 @@ def plot_graph_route(
         Size of the origin and destination nodes.
     ax
         If not None, plot on this pre-existing axes instance.
-    pg_kwargs
+    **pg_kwargs
         Keyword arguments to pass to `plot_graph`.
 
     Returns
@@ -394,7 +394,7 @@ def plot_graph_routes(
     route_linewidths
         If float, the one linewidth for all routes. Otherwise, the linewidth
         for each route.
-    pgr_kwargs
+    **pgr_kwargs
         Keyword arguments to pass to `plot_graph_route`.
 
     Returns
@@ -487,7 +487,7 @@ def plot_figure_ground(
         Fallback width, in pixels, for any street type not in `street_widths`.
     color
         The color of the streets.
-    pg_kwargs
+    **pg_kwargs
         Keyword arguments to pass to `plot_graph`.
 
     Returns

osmnx/projection.py

@@ -31,7 +31,7 @@ def is_projected(crs: Any) -> bool:  # noqa: ANN401
     Returns
     -------
     projected
-        True if `crs` is projected, otherwise False
+        True if `crs` is projected, otherwise False.
     """
     return bool(gpd.GeoSeries(crs=crs).crs.is_projected)
 

osmnx/routing.py

@@ -316,7 +316,7 @@ def shortest_path(
     Parameters
     ----------
     G
-        Input graph,
+        Input graph.
     orig
         Origin node ID(s).
     dest

pyproject.toml

@@ -69,6 +69,9 @@ strict = true
 warn_no_return = true
 warn_unreachable = true
 
+[tool.numpydoc_validation]
+checks = ["all", "ES01", "EX01", "GL08", "PR04", "RT03", "SA01"]
+
 [tool.ruff]
 cache-dir = "~/.cache/pre-commit/ruff"
 exclude = ["build/*"]