Skip to content
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

Add support for retrieving File Search result content in Run Steps and other fixes #294

Merged
merged 2 commits into from
Nov 20, 2024

Conversation

joseharriaga
Copy link
Collaborator

Features added

  • OpenAI.Assistants:
    • Added a Content property to RunStepFileSearchResult (step_details.tool_calls.file_search.results.content in the REST API).
      • When using an Assistant with the File Search tool, you can use this property to retrieve the contents of the File Search results that were used by the model.
    • Added FileSearchRankingOptions and FileSearchResults properties to RunStepDetailsUpdate.

Breaking Changes in Preview APIs

  • OpenAI.RealtimeConversation:
    • Renamed the From*() factory methods on ConversationContentPart to Create*Part() for consistency.
    • Removed an extraneous toolCallId parameter from ConversationItem.CreateSystemMessage().
  • OpenAI.Assistants:
    • Renamed RunStepType to RunStepKind.
    • Changed RunStepKind from an "extensible enum" to a regular enum.
    • Renamed the ToolCallId property of RunStepToolCall to Id.
    • Renamed the ToolKind property of RunStepToolCall to Kind.
    • Replaced the FileSearchRanker and FileSearchScoreThreshold properties of RunStepToolCall with a new FileSearchRankingOptions property that contains both values to make it clearer how they are related.

Bugs fixed

  • OpenAI.RealtimeConversation:
    • Fixed serialization issues with ConversationItem creation of system and assistant messages.

@joseharriaga joseharriaga merged commit bf3f0ed into main Nov 20, 2024
1 check passed
@joseharriaga joseharriaga deleted the joseharriaga/FileSearchResultContent branch November 20, 2024 22:50
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.

2 participants