-
Notifications
You must be signed in to change notification settings - Fork 176
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: Support DSPy #1166
Comments
@nirga I would like to contribute to this. I did check the contribution guidelines but am looking for more instructions on how to contribute - can you please help me? |
@parthmshah1302 sorry for the delay here - would love to assist you! You're welcome to join our community slack and we can help guide you. |
Hi @nirga Somesh here, can you assign this issue to me I'll start working on it asap |
@someshfengde you got it! |
Hi @nirga , I'm trying to figure out what exactly do I have to implement. From what I've understood till now we have to create a wrapper so that it can capture LLM calls occurs during the dspy evaluate function call. But evaluation method directly doesn't use LLM calls. calls were being made by program via Adding wrapper over Evaluate will not capture all calls but adding over predict will capture all LLM calls. Let me know what you think? Thanks. |
Hey @someshfengde! Sorry for the delay here as I was traveling. I see you're on our slack channel so I'll continue the discussion there! |
Hi @nirga , if no one else is working on this. Can you assign this to me! |
Hi @pranaybattu I'm working on it but still facing complexities with wrapping dspy library. If you want to collaborate let me know. :) |
Sure @someshfengde, I'd love to collaborate. Just let me know where you're stuck, and we can dive into it together! |
Are you available on slack? let's continue over there? |
Which component is this feature for?
All Packages
π Feature description
Instrument DSPy framework
π€ Why is this feature needed ?
βοΈ How do you aim to achieve this?
Similar to our LlamaIndex & LangChain instrumentations
ποΈ Additional Information
No response
π Have you spent some time to check if this feature request has been raised before?
Are you willing to submit PR?
None
The text was updated successfully, but these errors were encountered: