-
Notifications
You must be signed in to change notification settings - Fork 32
Open
Description
Originally posted by @dangotbanned in #124
Downstream in narwhals
, I've got a PR open which is trying to adjust to the new stubs:
From #96 the issues we have seem to stem from (236713a) and (f5d4e42) after which we get a lot of false negatives on:
ConstantExpression
LambdaExpression
DuckDBPyType
I think the fixes I have for ConstantExpression
+ LambdaExpression
would be pretty minor.
There's 1 or 2 examples for ConstantExpression
in the test suite already 😉
DuckDBPyType
might be a little more involved, since it's an issue with children
.
Also the way I've worked around it in narwhals
probably wouldn't be the same thing I'd write in the stubs
Metadata
Metadata
Assignees
Labels
No labels