-
Notifications
You must be signed in to change notification settings - Fork 90
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Polars deprecation inbound #894
Comments
thanks for the report! the deprecation seems to actually be from pandas because the current version of the tablevectorizer just transforms polars dataframes to pandas, which will change after #877 . Also after that PR |
here is a minimal reproducer, to add as a regression test: import polars as pl
from skrub import TableVectorizer
df = pl.DataFrame(dict(a=[0, 1], b=['a', 'b']))
TableVectorizer().fit_transform(df) |
(the problem seems to be that |
checked that this is fixed after #902 |
Describe the bug
I was working on a dataset with skrub together with polars and I noticed this warning pop up while running the
TableVectorizer
.Figured I'd mention it here just as a ping. It seems like something that may break in the future. If folks appreciate it I can also have a deeper look.
Steps/Code to Reproduce
Expected Results
No warning would be swell.
Actual Results
Mentioned above.
Versions
The text was updated successfully, but these errors were encountered: