-
Notifications
You must be signed in to change notification settings - Fork 7.8k
pass extra headers to LiteLLM #11360
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
base: main
Are you sure you want to change the base?
Conversation
…teLLM completion and acompletion (async/streaming); add unit tests\n\nCo-authored-by: openhands <openhands@all-hands.dev>
Coverage reportClick to see where and how coverage changed
This report was generated by python-coverage-comment-action |
…args/_partial_kwargs indirection\n- Pass extra_headers directly via kwargs to partial() calls\n\nCo-authored-by: openhands <openhands@all-hands.dev>
… into helper - Merge extra headers directly into kwargs across sync, async, streaming paths - Add LLM._get_extra_headers() to de-duplicate env parsing - Replace inline parsing in AsyncLLM and StreamingLLM - Run pre-commit and unit tests Co-authored-by: openhands <openhands@all-hands.dev>
Hi! I started running the integration tests on your PR. You will receive a comment with the results shortly. |
Looks like there are a few issues preventing this PR from being merged!
If you'd like me to help, just leave a comment, like
Feel free to include any additional details that might help me get this PR into a better state. You can manage your notification settings |
Summary
Changes
How to use
Verification
Backward compatibility
Related Issues
Notes
@enyst can click here to continue refining the PR
To run this PR locally, use the following command:
GUI with Docker:
CLI with uvx: