Remove content= from all SystemMessage(content="...") and UserMessage(content="..."),
#39544
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Fixed dead links in pypi and makie it more searchable * fix analysis
* multiagent updates * fix spelling error * adding missing file --------- Co-authored-by: Marko Hietala <[email protected]>
* Comply with the new API version * Fix * Fix linter * Fix linter * Fix
…zure-ai-inference clients you get from azure-ai-project client (#39348)
* Create sample for agents with logic apps * address comments * Update README and Prerequisites for samples * fix link validation issue * fix broken reference
* adding a sample how to add custom attributes to traces * updating comments * fixing tool check errors --------- Co-authored-by: Marko Hietala <[email protected]>
* add fixes to file/vs patches * fix trailing whitespace
dargilco
requested review from
luigiw,
needuv,
achauhan-scc,
kingernupur,
paulshealy1,
singankit,
jhakulin and
jayesh-tanna
as code owners
nick863
approved these changes
Feb 4, 2025
|
API change check APIView has identified API level changes in this PR and created following API reviews. |
singankit
approved these changes
Feb 4, 2025
w-javed
approved these changes
Feb 4, 2025
* update openapi sample and tool * update sync and aio methods for create vector store
* [AI] [Agents] Add sharepoint tool sample * Update sample_agents_sharepoint.py * Update sample_agents_sharepoint.py
…39589) * Record the tests for functions and write how to deploy azure function in the readme. * Describe creation of Azure function for python * Fix * Fix typos * Fix
* adding function tracing conveniency * updated documentation * reformta * update * change to restart ci * updating implementation * updating samples * adding unit tests and fixes * removing renamed file * adding unit test files * adding safeguard for the decorator to be no-op when otel not available * formatting changes done by black tool * pylint fixes * mypy fixes * fix a few tool errors * adding exception qualname to exception trace * updating change log --------- Co-authored-by: Marko Hietala <[email protected]>
… into dargilco/azure-ai-projects-pr-02-03-1607
rodex11
approved these changes
Feb 15, 2025
|
Hi @dargilco. Thank you for your interest in helping to improve the Azure SDK experience and for your contribution. We've noticed that there hasn't been recent engagement on this pull request. If this is still an active work stream, please let us know by pushing some changes or leaving a comment. Otherwise, we'll close this out in 7 days. |
github-actions
bot
added
the
no-recent-activity
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Remove
content=from allSystemMessage(content="...")andUserMessage(content="..."), in all samples and tests that do acompletecall using the Inference SDK. This keeps them short, and aligned with the new samples in the Inference package. But both options are supported.