-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
dspyComponent module only allows for a dspy.Predict module as input #1300
Comments
Ah good catch. It looks like this some recent updates (switching CoT to become a module) broke compatibility with the llamaindex integraton. Maybe we should pin the notebook to an old DSPy version? |
Sure! I was on dspy-ai 2.4.11 before this I think - the update just before! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I was following the llamaindex cookbook and reached an error for the code
The error trace showed that there was a pydantic validation error for "predict_module". That's because the DSPyComponent object only accepts a dspy.Predict(signature) input instead of a ChainOfThought object as listed in the cookbook.
I didn't have any errors running this cell before until I upgraded my dspy-ai version. Perhaps loosen the allowable classes for QueryComponent (the parent of DSPyComponent)? Or is the team's recommendation to use dspy.Predict instead?
Validation error trace
The text was updated successfully, but these errors were encountered: