-
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
Dataset destroys Example.input_keys values #898
Comments
Hi @jsleight , thanks for raising this. Currently, the behavior lies in declaring your Dataset type first and then setting the inputs - example from intro.ipynb:
but it does make sense to me to have |
I might have some time to make a PR. I can envision a couple of approaches so interested to see which you'd prefer.
|
arnavsinghvi11
added a commit
that referenced
this issue
Jun 17, 2024
Fix the issue of handling input_keys using Dataset class (Issue #898)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Minimal example (on dspy v2.4.0):
Expected to have the input_keys persist through the Dataset object. This line seems to be the problem.
The text was updated successfully, but these errors were encountered: