-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
[Feature Request]: Add Excel File support out of the box. #1488
Comments
Hey @vinaayakha, please go ahead. This would be a wonderful addition to Phidata! |
Hey @manthanguptaa, Thanks for assigning it to me. I will be able to complete it in 8 working days, and I will start at in Dec 01 2024. |
Hey @vinaayakha, how is the PR coming along? |
Hi @manthanguptaa, I've added the necessary changes to the tool, I still want to test it some, Due to my work, I haven't been able to do it. I hope to raise a PR this weekend for this change request. Thanks. |
Hi @manthanguptaa, Can i add the library unstructured==0.16.11? It has a lot of working tools that is required for excel parsing. |
Sure @vinaayakha, but when you try to import it put it under a try-except block. For example
Don't update the |
Hey @vinaayakha any update on this? Are you still working on it? |
CSV is cool, but as I process a lot of Excel, I created a custom model but thought it would be great to have it out of the box in the phi data framework to dump a data frame from Excel.
Should I raise a PR for review?
The text was updated successfully, but these errors were encountered: