litellm
litellm copied to clipboard
Update support for langfuse metadata
Title
Update support for langfuse metadata
Relevant issues
Builds upon #3221
Type
๐ New Feature ๐งน Refactoring ๐ Documentation โ Test
Changes
- Added ability to set trace release, version, metadata
- Added ability to update fields during a trace continuation
- Added ability to update input and output during a trace continuation
- Wrote new test for verifying metadata is set correctly
- Small improvement to setting secret boolean, prevent unnecessary literal_eval
- Small improvements to langfuse tests
Testing
- Ran existing test suite along with new test
Notes
- Updates to support most langfuse metadata facilities
Pre-Submission Checklist (optional but appreciated):
- [x] I have included relevant documentation updates (stored in /docs/my-website)
OS Tests (optional but appreciated):
- [ ] Tested on Windows
- [ ] Tested on MacOS
- [x] Tested on Linux
The latest updates on your projects. Learn more about Vercel for Git โ๏ธ
Name | Status | Preview | Comments | Updated (UTC) |
---|---|---|---|---|
litellm | โ Ready (Inspect) | Visit Preview | ๐ฌ Add feedback | May 5, 2024 6:57pm |
@alexanderepstein added some comments.
If we can address those + attach a screenshot of the langfuse testing passing for you - i think this should be good to merge
I am not sure I see your comments, let me know if I'm missing something.
Had to add mark one test as skippable if you don't have an OpenAI api key.
Here is the test output for the file.
are you not able to see these? @alexanderepstein
Looks like your comments are pending https://github.com/orgs/community/discussions/10369
I attempted to address the comments I can see in your screenshot.