-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Feat(dspy): from_pandas support #1176
Conversation
PS: Just for the ruff checks, I did additional changes on types. Let me know if I need to remove that. |
very nice update ! |
Merge from main
I see one else block removed in |
Okay, so I reverted to the implementation along with I added the Let me know, I will be doing that in other of my PRs too. |
Thanks for the contribution!! Did you try running |
Let's merge this if @krypticmouse approves |
Hey thanks for the quick pointer, although I have used this command earlier which gives me this output:
To solve now, I kinda applied a small hack (reverting back to the contents in the main branch and adding the from_pandas function) However, I am guessing I might need to set the environment once again to not see logs like above when doing ruff fix? |
Yea try syncing the repo with the current version and lemme know if you still see the errors |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm 🚢
Thank you for the contribution. Merging it! |
This PR adds support for to load dspy dataset from dataframes directly. Something which I personally found useful when fetching from some sources did some cleanup and get dspy dataset without saving it as a csv.
Fixes issue: #1177