docs(python): Address ignored Ruff doc rules

py-polars/docs/source/conf.py

@@ -235,14 +235,14 @@ def _minify_classpaths(s: str) -> str:
     )
 
 
-def process_signature(app, what, name, obj, opts, sig, ret):
+def process_signature(app, what, name, obj, opts, sig, ret):  # noqa: D103
     return (
         _minify_classpaths(sig) if sig else sig,
         _minify_classpaths(ret) if ret else ret,
     )
 
 
-def setup(app):
+def setup(app):  # noqa: D103
     # TODO: a handful of methods do not seem to trigger the event for
     #  some reason (possibly @overloads?) - investigate further...
     app.connect("autodoc-process-signature", process_signature)

py-polars/polars/dataframe/frame.py

@@ -3306,9 +3306,9 @@ def write_delta(
         ... )  # doctest: +SKIP
 
         """
-        from polars.io.delta import check_if_delta_available, resolve_delta_lake_uri
+        from polars.io.delta import _check_if_delta_available, _resolve_delta_lake_uri
 
-        check_if_delta_available()
+        _check_if_delta_available()
 
         from deltalake.writer import (
             try_get_deltatable,
@@ -3319,7 +3319,7 @@ def write_delta(
             delta_write_options = {}
 
         if isinstance(target, (str, Path)):
-            target = resolve_delta_lake_uri(str(target), strict=False)
+            target = _resolve_delta_lake_uri(str(target), strict=False)
 
         unsupported_cols = {}
         unsupported_types = [Time, Categorical, Null]

py-polars/polars/dataframe/groupby.py

@@ -845,6 +845,19 @@ def agg(
         *aggs: IntoExpr | Iterable[IntoExpr],
         **named_aggs: IntoExpr,
     ) -> DataFrame:
+        """
+        Compute aggregations for each group of a groupby operation.
+
+        Parameters
+        ----------
+        *aggs
+            Aggregations to compute for each group of the groupby operation,
+            specified as positional arguments.
+            Accepts expression input. Strings are parsed as column names.
+        **named_aggs
+            Additional aggregations, specified as keyword arguments.
+            The resulting columns will be renamed to the keyword used.
+        """
         return (
             self.df.lazy()
             .groupby_rolling(
@@ -1046,6 +1059,19 @@ def agg(
         *aggs: IntoExpr | Iterable[IntoExpr],
         **named_aggs: IntoExpr,
     ) -> DataFrame:
+        """
+        Compute aggregations for each group of a groupby operation.
+
+        Parameters
+        ----------
+        *aggs
+            Aggregations to compute for each group of the groupby operation,
+            specified as positional arguments.
+            Accepts expression input. Strings are parsed as column names.
+        **named_aggs
+            Additional aggregations, specified as keyword arguments.
+            The resulting columns will be renamed to the keyword used.
+        """
         return (
             self.df.lazy()
             .groupby_dynamic(

py-polars/polars/datatypes/classes.py

@@ -3,7 +3,7 @@
 import contextlib
 from datetime import timezone
 from inspect import isclass
-from typing import TYPE_CHECKING, Any, Callable, Iterator, Mapping, Sequence
+from typing import TYPE_CHECKING, Any, Callable, Iterable, Iterator, Mapping, Sequence
 
 import polars.datatypes
 
@@ -31,7 +31,7 @@ def __init__(self, method: Callable[..., Any] | None = None) -> None:
     def __get__(self, instance: Any, cls: type | None = None) -> Any:
         return self.fget(cls)  # type: ignore[misc]
 
-    def getter(self, method: Callable[..., Any]) -> Any:
+    def getter(self, method: Callable[..., Any]) -> Any:  # noqa: D102
         self.fget = method
         return self
 
@@ -46,25 +46,29 @@ def _string_repr(cls) -> str:
         return _dtype_str_repr(cls)
 
     def base_type(cls) -> PolarsDataType:
+        """Return the base type."""
         return cls
 
     @classproperty
     def is_nested(self) -> bool:
+        """Check if this data type is nested."""
         return False
 
     @classmethod
     def is_(cls, other: PolarsDataType) -> bool:
+        """Check if this DataType is the same as another DataType."""
         return cls == other and hash(cls) == hash(other)
 
     @classmethod
     def is_not(cls, other: PolarsDataType) -> bool:
+        """Check if this DataType is NOT the same as another DataType."""
         return not cls.is_(other)
 
 
 class DataType(metaclass=DataTypeClass):
     """Base class for all Polars data types."""
 
-    def __new__(cls, *args: Any, **kwargs: Any) -> PolarsDataType:  # type: ignore[misc]
+    def __new__(cls, *args: Any, **kwargs: Any) -> PolarsDataType:  # type: ignore[misc]  # noqa: D102
         # this formulation allows for equivalent use of "pl.Type" and "pl.Type()", while
         # still respecting types that take initialisation params (eg: Duration/Datetime)
         if args or kwargs:
@@ -95,6 +99,7 @@ def base_type(cls) -> DataTypeClass:
 
     @classproperty
     def is_nested(self) -> bool:
+        """Check if this data type is nested."""
         return False
 
     @classinstmethod  # type: ignore[arg-type]
@@ -158,15 +163,30 @@ def _custom_reconstruct(
 
 
 class DataTypeGroup(frozenset):  # type: ignore[type-arg]
+    """Group of data types."""
+
     _match_base_type: bool
 
-    def __new__(cls, items: Any, *, match_base_type: bool = True) -> DataTypeGroup:
+    def __new__(
+        cls, items: Iterable[DataType | DataTypeClass], *, match_base_type: bool = True
+    ) -> DataTypeGroup:
+        """
+        Construct a DataTypeGroup.
+
+        Parameters
+        ----------
+        items :
+            iterable of data types
+        match_base_type:
+            match the base type
+
+        """
         for it in items:
             if not isinstance(it, (DataType, DataTypeClass)):
                 raise TypeError(
                     f"DataTypeGroup items must be dtypes; found {type(it).__name__!r}"
                 )
-        dtype_group = super().__new__(cls, items)
+        dtype_group = super().__new__(cls, items)  # type: ignore[arg-type]
         dtype_group._match_base_type = match_base_type
         return dtype_group
 
@@ -201,6 +221,7 @@ class NestedType(DataType):
 
     @classproperty
     def is_nested(self) -> bool:
+        """Check if this data type is nested."""
         return True
 
 
@@ -406,6 +427,8 @@ class Unknown(DataType):
 
 
 class List(NestedType):
+    """Nested list/array type with variable length of inner lists."""
+
     inner: PolarsDataType | None = None
 
     def __init__(self, inner: PolarsDataType | PythonDataType):
@@ -466,6 +489,8 @@ def __repr__(self) -> str:
 
 
 class Array(NestedType):
+    """Nested list/array type with fixed length of inner arrays."""
+
     inner: PolarsDataType | None = None
     width: int
 
@@ -524,6 +549,8 @@ def __repr__(self) -> str:
 
 
 class Field:
+    """Definition of a single field within a `Struct` DataType."""
+
     def __init__(self, name: str, dtype: PolarsDataType):
         """
         Definition of a single field within a `Struct` DataType.
@@ -551,6 +578,8 @@ def __repr__(self) -> str:
 
 
 class Struct(NestedType):
+    """Struct composite type."""
+
     def __init__(self, fields: Sequence[Field] | SchemaDict):
         """
         Struct composite type.

py-polars/polars/datatypes/convert.py

@@ -73,7 +73,7 @@
 T = TypeVar("T")
 
 
-def cache(function: Callable[..., T]) -> T:
+def cache(function: Callable[..., T]) -> T:  # noqa: D103
     # need this to satisfy mypy issue with "@property/@cache combination"
     # See: https://github.com/python/mypy/issues/5858
     return functools.lru_cache()(function)  # type: ignore[return-value]
@@ -98,7 +98,10 @@ def cache(function: Callable[..., T]) -> T:
 
 
 @functools.lru_cache(16)
-def map_py_type_to_dtype(python_dtype: PythonDataType | type[object]) -> PolarsDataType:
+def _map_py_type_to_dtype(
+    python_dtype: PythonDataType | type[object],
+) -> PolarsDataType:
+    """Convert Python data type to Polars data type."""
     if python_dtype is float:
         return Float64
     if python_dtype is int:
@@ -134,14 +137,14 @@ def map_py_type_to_dtype(python_dtype: PythonDataType | type[object]) -> PolarsD
     if hasattr(python_dtype, "__origin__") and hasattr(python_dtype, "__args__"):
         base_type = python_dtype.__origin__
         if base_type is not None:
-            dtype = map_py_type_to_dtype(base_type)
+            dtype = _map_py_type_to_dtype(base_type)
             nested = python_dtype.__args__
             if len(nested) == 1:
                 nested = nested[0]
             return (
                 dtype
                 if nested is None
-                else dtype(map_py_type_to_dtype(nested))  # type: ignore[operator]
+                else dtype(_map_py_type_to_dtype(nested))  # type: ignore[operator]
             )
 
     raise TypeError("Invalid type")
@@ -424,7 +427,7 @@ def py_type_to_dtype(
         if is_polars_dtype(data_type):
             return data_type
     try:
-        return map_py_type_to_dtype(data_type)
+        return _map_py_type_to_dtype(data_type)
     except (KeyError, TypeError):  # pragma: no cover
         if not raise_unmatched:
             return None

py-polars/polars/expr/datetime.py

@@ -875,12 +875,39 @@ def ordinal_day(self) -> Expr:
         return wrap_expr(self._pyexpr.dt_ordinal_day())
 
     def time(self) -> Expr:
+        """
+        Extract time.
+
+        Applies to Datetime columns only; fails on Date.
+
+        Returns
+        -------
+        Time
+        """
         return wrap_expr(self._pyexpr.dt_time())
 
     def date(self) -> Expr:
+        """
+        Extract date from date(time).
+
+        Applies to Date and Datetime columns.
+
+        Returns
+        -------
+        Date
+        """
         return wrap_expr(self._pyexpr.dt_date())
 
     def datetime(self) -> Expr:
+        """
+        Return datetime.
+
+        Applies to Datetime columns.
+
+        Returns
+        -------
+        Datetime
+        """
         return wrap_expr(self._pyexpr.dt_datetime())
 
     def hour(self) -> Expr:

py-polars/polars/io/csv/batched_reader.py

@@ -26,6 +26,8 @@
 
 
 class BatchedCsvReader:
+    """Read a CSV file in batches."""
+
     def __init__(
         self,
         source: str | Path,

py-polars/polars/io/delta.py

@@ -126,7 +126,7 @@ def read_delta(
     if pyarrow_options is None:
         pyarrow_options = {}
 
-    resolved_uri = resolve_delta_lake_uri(source)
+    resolved_uri = _resolve_delta_lake_uri(source)
 
     dl_tbl = _get_delta_lake_table(
         table_path=resolved_uri,
@@ -254,7 +254,7 @@ def scan_delta(
     if pyarrow_options is None:
         pyarrow_options = {}
 
-    resolved_uri = resolve_delta_lake_uri(source)
+    resolved_uri = _resolve_delta_lake_uri(source)
     dl_tbl = _get_delta_lake_table(
         table_path=resolved_uri,
         version=version,
@@ -266,7 +266,7 @@ def scan_delta(
     return scan_pyarrow_dataset(pa_ds)
 
 
-def resolve_delta_lake_uri(table_uri: str, strict: bool = True) -> str:
+def _resolve_delta_lake_uri(table_uri: str, strict: bool = True) -> str:
     parsed_result = urlparse(table_uri)
 
     resolved_uri = str(
@@ -297,7 +297,7 @@ def _get_delta_lake_table(
     DeltaTable
 
     """
-    check_if_delta_available()
+    _check_if_delta_available()
 
     if delta_table_options is None:
         delta_table_options = {}
@@ -312,7 +312,7 @@ def _get_delta_lake_table(
     return dl_tbl
 
 
-def check_if_delta_available() -> None:
+def _check_if_delta_available() -> None:
     if not _DELTALAKE_AVAILABLE:
         raise ImportError(
             "deltalake is not installed. Please run `pip install deltalake>=0.9.0`."

py-polars/polars/utils/convert.py

@@ -30,13 +30,13 @@
     elif _ZONEINFO_AVAILABLE:
         from backports.zoneinfo._zoneinfo import ZoneInfo
 
-    def get_zoneinfo(key: str) -> ZoneInfo:
+    def get_zoneinfo(key: str) -> ZoneInfo:  # noqa: D103
         pass
 
 else:
 
     @lru_cache(None)
-    def get_zoneinfo(key: str) -> ZoneInfo:
+    def get_zoneinfo(key: str) -> ZoneInfo:  # noqa: D103
         return zoneinfo.ZoneInfo(key)
 
 

py-polars/polars/utils/various.py

@@ -320,7 +320,7 @@ def str_duration_(td: str | None) -> int | None:
 NS = TypeVar("NS")
 
 
-class sphinx_accessor(property):
+class sphinx_accessor(property):  # noqa: D101
     def __get__(  # type: ignore[override]
         self,
         instance: Any,

py-polars/pyproject.toml

@@ -143,12 +143,9 @@ ignore = [
   # pycodestyle
   # TODO: Remove errors below to further improve docstring linting
   # Ordered from most common to least common errors.
-  "D105",
-  "D100",
-  "D103",
-  "D102",
-  "D104",
-  "D101",
+  "D105", # Missing docstring in magic method
+  "D100", # Missing docstring in public module
+  "D104", # Missing docstring in public package
 ]
 
 [tool.ruff.pycodestyle]