logfire icon indicating copy to clipboard operation
logfire copied to clipboard

[Docs] - Update LiteLLM Integration Docs

Open ishaan-jaff opened this issue 1 year ago • 5 comments

Hi i'm the LiteLLM (https://github.com/BerriAI/litellm) maintainer. Made some improvements to the doc section on using litellm with logfire

ishaan-jaff avatar Jul 24 '24 15:07 ishaan-jaff

@Kludex @alexmojaki can I get a review on this PR ?

ishaan-jaff avatar Jul 24 '24 15:07 ishaan-jaff

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (c763236) to head (f4e7881). Report is 73 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #333   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          119       119           
  Lines         8882      8885    +3     
  Branches      1157      1159    +2     
=========================================
+ Hits          8882      8885    +3     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Jul 24 '24 15:07 codecov[bot]

Let me know when you have applied the comments @ishaan-jaff 🙏

Kludex avatar Jul 29 '24 01:07 Kludex

I thought LiteLLM was a library to give introp between different LLMs, why does it need a proxy?

samuelcolvin avatar Jul 29 '24 17:07 samuelcolvin

@ishaan-jaff are you still interested here? If so, please apply comments, and reply the question above.

Kludex avatar Aug 07 '24 06:08 Kludex

@ishaan-jaff Happy to continue this PR if you have time to reply the comments here. 🙏

For now, I'll be closing this PR.

Kludex avatar Sep 26 '24 08:09 Kludex