-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
System Prompt Does Not Utilize dspy.Signature
Docstring (__doc__
)
#1836
Comments
Hey @yanmxa ! Thank you so much for opening an issue & PR for this. I think you're observing this in ReAct only, not in general. The issue can be addressed by editing predict/react.py |
We need to pass instructions to this constructor: Line 67 in 698d0ac
|
Thanks, @okhat! You got it! This issue occurs when using the |
Thank you, @yanmxa ! A PR would be welcome :D The change should be a one-liner but you may need to check that it does work. For line 67, it should become: dspy.Signature({**signature.input_fields, **signature.output_fields}, signature.instructions) |
@okhat Thank you for the prompt response! |
The
dspy.Signature
class docstring (doc) is not dynamically parsed or integrated into the system prompt.Steps to Reproduce:
Current Behavior:
The system prompt does not include the docstring from the KubeEngineer class.
Expected Output:
The text was updated successfully, but these errors were encountered: