Skip to content

(fix) LiteLLM Proxy fix GET /files/{file_id:path}/content" endpoint - #7342

Merged
ishaan-jaff merged 10 commits into
mainfrom
litellm_fix_batches
Dec 21, 2024
Merged

ishaan-jaff merged 10 commits into
mainfrom
litellm_fix_batches

Conversation

@ishaan-jaff

@ishaan-jaff ishaan-jaff commented Dec 21, 2024 •

Copy link
Copy Markdown
Contributor

Fix GET /files/{file_id:path}/content" endpoint

Relevant issues

Bug occurred when trying to do this

 content = client.files.content(batch.output_file_id)
 content.write_to_file(output_path)

Type

🆕 New Featurec
✅ Test

Test cases added

  • e2e tests with OpenAI Python SDK - creating batches + reading output files
  • e2e tests with OpenAI Python SDK - running all file operations

Changes

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

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

@vercel

vercel Bot commented Dec 21, 2024 •

Copy link
Copy Markdown

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 Dec 21, 2024 5:28am

@ishaan-jaff
ishaan-jaff merged commit b90b98b into main Dec 21, 2024
ishaan-jaff added a commit that referenced this pull request Dec 21, 2024
…#7342)

* fix order of get_file_content

* update e2 files tests

* add e2 batches endpoint testing

* update config.yml

* write content to file

* use correct oai_misc_config

* fixes for openai batches endpoint testing

* remove extra out file

* fix input.jsonl
@ishaan-berri
ishaan-berri deleted the litellm_fix_batches branch March 26, 2026 21:53
fzowl pushed a commit to fzowl/litellm that referenced this pull request Jun 24, 2026
…BerriAI#7342)

* fix order of get_file_content

* update e2 files tests

* add e2 batches endpoint testing

* update config.yml

* write content to file

* use correct oai_misc_config

* fixes for openai batches endpoint testing

* remove extra out file

* fix input.jsonl

This branch was successfully deployed

1 active deployment
Preview — f25544e2 Deployed Dec 21, 2024 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant