Skip to content
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

support polars #3921

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

celestinoxp
Copy link
Contributor

This pull request introduces an initial solution for handling Polars integration into PyCaret code. Initially, completely replacing the code to use Polars natively may seem like a complex and laborious task. As an intermediate approach, this pull request proposes adding conversion capabilities from DataFrame Polars to DataFrame Pandas at certain points in the code.

Converting a dataset from Polars to Pandas is a workaround that allows PyCaret to work with datasets provided in Polars. This approach offers a viable way to continue Polars integration development and exploration without disrupting your existing workflow.

We welcome any feedback or suggestions to improve this solution and are open to discussing more efficient and robust alternatives for integrating Polars with PyCaret.

Related Issue: #3919

celestinoxp and others added 3 commits February 26, 2024 18:13
support polars detecting and converting dataframe to pandas

-added a test in test_datasets.py
@Yard1
Copy link
Member

Yard1 commented Mar 3, 2024

Wouldn't converting to pandas remove any benefit from using polars in the first place?

@celestinoxp
Copy link
Contributor Author

Wouldn't converting to pandas remove any benefit from using polars in the first place?

i can work on this, but i'm not sure how can we do... any ideas?

@celestinoxp
Copy link
Contributor Author

@keitazoumana Can you help us know how we can use dataset type checking without having to repeat thousands of lines of code? Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants