feature(dspy): dspy-integration-with-langfuse #1186
Conversation
xucailiang
changed the title
xucailiang
changed the title
There was a problem hiding this comment.
Hey @xucailiang I have left some comments. I am learning DSPy too and just wondering whether the proposed integration could be done in a more elegant way
…e(dspy)-dspy-integration-with-langfuse
I have rewritten the code, the new calling method is more elegant, I realize that all LLM reasoning can be collected to langfuse, but it is difficult for me to test all the code. I have already tested my submission, maybe we can continue to integrate other LLM modules after this PR is merged |
…e(dspy)-dspy-integration-with-langfuse
…e(dspy)-dspy-integration-with-langfuse # Conflicts: # dsp/modules/azure_openai.py
…e(dspy)-dspy-integration-with-langfuse
|
@xucailiang hey! I will check it out this weekend |
Thanks for your time, I'm not sure if we'll wrap up this pull request over the weekend. If you have any suggestions or ideas to improve this, please feel free to share them with me at your earliest convenience. I'd really appreciate it! |
hi，I wonder if this discussion is still ongoing? |
|
+1, also interested in this! |
Hi! Unfortunately I have not had time to run it locally yet. I will do it. However, I am not a DSPy maintainer, just another user, so all my comments are just my comments :) I wonder whether it would make sense to ping one of the contributers so that they can take a look as well |
|
@arnavsinghvi11 hi,I was wondering if you could review this PR so I can continue integrating langfuse on other modules.Thank you. |
…e(dspy)-dspy-integration-with-langfuse
As mentioned in issue #1179 , I wrote some code to integrate langfuse.
when configured correctly, It will support the display of data from OpenAI, AzureOpenAI and Ollama on the Langfuse dashboard