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

No need to write "content=" when creating UserMessage, SystemMessage, AssistantMessage and ToolMessage #39130

Conversation

dargilco
Copy link
Member

@dargilco dargilco commented Jan 11, 2025

Changes:

  • No need to write content= when creating UserMessage, SystemMessage, AssistantMessage and ToolMessage. Add unit-tests. Update all samples. Update some tests.
  • Updated CHANGELOG.md
    • To mention the above
    • Bug fix per Liudmila's merged PR to fix a tracing issue
  • Rename 3 samples to more accurately reflect what it shows (using "Dict" as positional input type to complete method)

@github-actions github-actions bot added the AI Model Inference Issues related to the client library for Azure AI Model Inference (\sdk\ai\azure-ai-inference) label Jan 11, 2025
@azure-sdk
Copy link
Collaborator

API change check

APIView has identified API level changes in this PR and created following API reviews.

azure-ai-inference

@dargilco dargilco self-assigned this Jan 13, 2025
@dargilco dargilco merged commit 40c143e into feature/azure-ai-inference-beta7 Jan 13, 2025
21 checks passed
@dargilco dargilco deleted the dargilco/azure-ai-inference-pr-01-10-1020 branch January 13, 2025 23:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AI Model Inference Issues related to the client library for Azure AI Model Inference (\sdk\ai\azure-ai-inference)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants