litellm icon indicating copy to clipboard operation
litellm copied to clipboard

Litellm dev 03 05 2025 contributor prs

Open krrishdholakia opened this issue 9 months ago โ€ข 3 comments

Title

Relevant issues

Type

๐Ÿ†• New Feature ๐Ÿ› Bug Fix ๐Ÿงน Refactoring ๐Ÿ“– Documentation ๐Ÿš„ Infrastructure โœ… Test

Changes

[REQUIRED] Testing - Attach a screenshot of any new tests passing locally

If UI changes, send a screenshot/GIF of working UI fixes

krrishdholakia avatar Mar 08 '25 16:03 krrishdholakia

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 Jul 4, 2025 7:18pm

vercel[bot] avatar Mar 08 '25 16:03 vercel[bot]

Is there some progress on adding GitHub Copilot to the list of providers? Will this branch be merged into main?

S1M0N38 avatar Apr 09 '25 16:04 S1M0N38

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Apr 22 '25 22:04 CLAassistant

hi @krrishdholakia is there any blockers preventing this PR merged?

nhs000 avatar May 09 '25 02:05 nhs000

image

(Joking of course, thanks for the work :heart: )

NINNiT avatar May 14 '25 07:05 NINNiT

@zzeniou86, This PR author is @krrishdholakia


@krrishdholakia, Would you merge main into this?

https://github.com/BerriAI/litellm/pull/11139#issuecomment-2911214280

SmartManoj avatar Jun 07 '25 21:06 SmartManoj

2 merge conflicts. Rename tests.litellm with tests.test_litellm

image

Accepting both changes

image

SmartManoj avatar Jun 08 '25 02:06 SmartManoj

@SmartManoj @krrishdholakia Any update on this?

I have a "build your own coding agent" post that uses this branch of LiteLLM: I would like to be able to simplify the prerequisites to just use a LiteLLM release instead of the branch.

I would also love to be able to use Github Copilot models in aider, but they are waiting on this litellm branch to be merged.

Today, I was experimenting with DSPy and discovered that it uses LiteLLM for language model access so I was able to use it like this:

#!/usr/bin/env python3

# Usage:
#   uv venv
#   source .venv/bin/activate
#   uv pip install dspy
#   uv pip install git+https://github.com/BerriAI/litellm.git@litellm_dev_03_05_2025_contributor_prs

import dspy
lm = dspy.LM('github_copilot/o3-mini', temperature = 1.0, max_tokens = 20000, extra_headers = {"editor-version": "vscode/1.85.1"})
print(lm("Say this is a test!"))

It would be great to not have to do workarounds to get Github Copilot working in projects that use LiteLLM.

Apart from resolving merge conflicts, is there anything preventing this from being merged into a new release?

Thanks for the great project!

kanaka avatar Jun 16 '25 18:06 kanaka

testing in https://github.com/BerriAI/litellm/pull/11786

SmartManoj avatar Jun 17 '25 02:06 SmartManoj

What is the current status of this? @SmartManoj

PythonTilk avatar Jun 27 '25 23:06 PythonTilk

Simply resolving like this is enough.

CC: @krrishdholakia @ishaan-jaff

SmartManoj avatar Jun 28 '25 02:06 SmartManoj

okay. is this getting merged into main anytime soon?

PythonTilk avatar Jul 02 '25 22:07 PythonTilk

Hi all - i'm on this !

ishaan-jaff avatar Jul 04 '25 19:07 ishaan-jaff

Well Nice

PythonTilk avatar Jul 04 '25 19:07 PythonTilk