Skip to content

Conversation

@codomposer
Copy link

Fixes

Closes #2641

  • I understand that this repository is auto-generated and my pull request may not be merged

Changes being requested

Additional context & links

@codomposer codomposer requested a review from a team as a code owner October 30, 2025 19:16
yashwantbezawada pushed a commit to yashwantbezawada/openai-python that referenced this pull request Nov 6, 2025
Fixes openai#2717

The poll() methods in both Files and FileBatches were passing IDs as
positional arguments instead of named parameters when calling retrieve().

This causes TypeError when retrieve() tries to match parameters, since
the method signature expects named parameters.

Fixed in 4 locations:
- FileBatches.poll() (sync)
- AsyncFileBatches.poll() (async)
- Files.poll() (sync)
- AsyncFiles.poll() (async)

Changed from positional to named parameters:
- batch_id, -> batch_id=batch_id,
- file_id, -> file_id=file_id,

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
yashwantbezawada pushed a commit to yashwantbezawada/openai-python that referenced this pull request Nov 6, 2025
Fixes openai#2717

The poll() methods in both Files and FileBatches were passing IDs as
positional arguments instead of named parameters when calling retrieve().

This causes TypeError when retrieve() tries to match parameters, since
the method signature expects named parameters.

Fixed in 4 locations:
- FileBatches.poll() (sync)
- AsyncFileBatches.poll() (async)
- Files.poll() (sync)
- AsyncFiles.poll() (async)

Changed from positional to named parameters:
- batch_id, -> batch_id=batch_id,
- file_id, -> file_id=file_id,
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The usage object for streaming doesn't support all the stats reported by LLMs

2 participants