-
Notifications
You must be signed in to change notification settings - Fork 170
Closed
Labels
daskIssue is related to dask backendIssue is related to dask backendduckdbIssue is related to duckdb backendIssue is related to duckdb backendinternalspark-like
Description
Originally posted by @dangotbanned in #2000 (comment)
Realised while reviewing (#2000) that we're not enforcing the newer-style protocols for most ofLazyExpr.
The one exception being LazyExprNameNamespace.
narwhals/narwhals/_compliant/expr.py
Lines 904 to 910 in 3d92e0d
| @property | |
| def name(self) -> LazyExprNameNamespace[Self]: | |
| return LazyExprNameNamespace(self) | |
| class _ExprNamespace( # type: ignore[misc] | |
| _StoresCompliant[CompliantExprT_co], Protocol[CompliantExprT_co] |
Everything else is Any, which IIRC was for the benefit of not_implemented 🤔
narwhals/narwhals/_compliant/expr.py
Lines 282 to 293 in 3d92e0d
| @property | |
| def str(self) -> Any: ... | |
| @property | |
| def name(self) -> Any: ... | |
| @property | |
| def dt(self) -> Any: ... | |
| @property | |
| def cat(self) -> Any: ... | |
| @property | |
| def list(self) -> Any: ... | |
| @property | |
| def struct(self) -> Any: ... |
Related
Metadata
Metadata
Assignees
Labels
daskIssue is related to dask backendIssue is related to dask backendduckdbIssue is related to duckdb backendIssue is related to duckdb backendinternalspark-like