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

Prepare 2.0.0 release (Part 1) #233

Merged
merged 1 commit into from
Sep 30, 2024
Merged

Prepare 2.0.0 release (Part 1) #233

merged 1 commit into from
Sep 30, 2024

Conversation

joseharriaga
Copy link
Collaborator

  • Implemented ChatMessageContent to encapsulate the representation of content parts in ChatMessage, ChatCompletion, and StreamingChatCompletionUpdate.
  • Changed the representation of function arguments to BinaryData in ChatToolCall, StreamingChatToolCallUpdate, ChatFunctionCall, and StreamingChatFunctionCallUpdate.
  • Renamed OpenAIClientOptions's ApplicationId to UserAgentApplicationId (commit_hash)
  • Renamed StreamingChatToolCallUpdate's Id to ToolCallId
  • Renamed StreamingChatCompletionUpdate's Id to CompletionId
  • Replaced Auto and None in the deprecated ChatFunctionChoice with CreateAutoChoice() and CreateNoneChoice()
  • Replaced the deprecated ChatFunctionChoice(ChatFunction) constructor with CreateNamedChoice(string functionName)
  • Renamed FileClient to OpenAIFileClient and the corresponding GetFileClient() method in OpenAIClient to GetOpenAIFileClient().
  • Renamed ModelClient to OpenAIModelClient and the corresponding GetModelClient() method in OpenAIClient to GetOpenAIModelClient().

@joseharriaga joseharriaga merged commit 31c2ba6 into main Sep 30, 2024
1 check passed
@joseharriaga joseharriaga deleted the joseharriaga/Pre-2.0.0 branch September 30, 2024 21:36
@RogerBarreto
Copy link
Contributor

RogerBarreto commented Oct 1, 2024

ChatMessageContent triggers a conflict with our Semantic Kernel ChatMessageContent type, this change causes a very big impact on all of our customers.

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.

3 participants