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.

Sign up for GitHub

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

Jump to bottom

[azopenai] Inference changes for 2024-10-01-preview #23672

Merged
merged 8 commits into from
Nov 12, 2024
Merged

[azopenai] Inference changes for 2024-10-01-preview #23672

merged 8 commits into from
Nov 12, 2024

Conversation

glecaros
Copy link
Contributor

  • The purpose of this PR is explained in this or a referenced issue.
  • The PR does not update generated files.
  • Tests are included and/or updated for code changes.
  • Updates to module CHANGELOG.md are included.
  • MIT license headers are included in each file.

richardpark-msft
richardpark-msft requested changes Oct 29, 2024
View reviewed changes
sdk/ai/azopenai/models.go Show resolved Hide resolved
sdk/ai/azopenai/models.go Show resolved Hide resolved
@richardpark-msft
Copy link
Member

@glecaros, the CI failures probably just mean the tests need to be re-recorded.

richardpark-msft
richardpark-msft reviewed Oct 31, 2024
View reviewed changes
sdk/ai/azopenai/models_extra.go Show resolved Hide resolved
sdk/ai/azopenai/client_extra.go Show resolved Hide resolved
sdk/ai/azopenai/CHANGELOG.md Show resolved Hide resolved
richardpark-msft
richardpark-msft approved these changes Oct 31, 2024
View reviewed changes
Copy link
Member

@richardpark-msft richardpark-msft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a comment and a changelog entry and I think it's good.

After this we'll want to submit a new APIView as well.

@glecaros
Copy link
Contributor Author

glecaros commented Nov 5, 2024

/azp run go - azopenai

@azure-pipelines Azure Pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@glecaros glecaros merged commit b6d204f into main Nov 12, 2024
14 checks passed
@glecaros glecaros deleted the glecaros/aoai_2024-10-01-preview branch November 12, 2024 18:33
glecaros added a commit that referenced this pull request Nov 19, 2024
@glecaros
[azopenai] Inference changes for 2024-10-01-preview (#23672)
1e6a91b
glecaros added a commit that referenced this pull request Dec 3, 2024
@glecaros @richardpark-msft
@azure-sdk @danieljurek @benbp @tadelesh @analogrelay @scbedd @hallipr @ashruti-msft @tanyasethi-msft @live1206 @gracewilcox @jliusan @mikeharder @weshaggard
[azopenaiassistants] Updates for 2024-10-01-preview (#23730)
2ea046b 
* hash+gen

* Adding in MessageAttachmentToolDefinition

* Accidental copy/paste

* Updating doc comment for MessageAttachmentToolDefinition

* Update docindex.yml to use Python 3.11 (#23719)

Co-authored-by: Daniel Jurek <djurek@microsoft.com>

* Refresh OIDC token in azadmin test resources scripts (#23722)

* remove pkg.go.dev link in readme (#23708)

* remove pkg.go.dev link in readme

* fix changelog

* upgrade typesspec-go (#23720)

* [azopenai] Inference changes for 2024-10-01-preview (#23672)

* update for 1.2.0 release (#23728)

* Sync eng/common directory with azure-sdk-tools for PR 9328 (#23692)

* ensure that ChangedServices value within the generated pr diff is always properly an array of strings
* upload the changed services as an attachment named AdditionalTags in save-package-properties.yml, to allow PipelineWitness to tag public builds during log processing

---------

Co-authored-by: Scott Beddall <scbedd@microsoft.com>
Co-authored-by: Scott Beddall <45376673+scbedd@users.noreply.github.com>
Co-authored-by: Patrick Hallisey <hallipr@gmail.com>

* refine typespec release tool (#23735)

* Added permissions & resourcetype parameters in listblob response  (#23687)

* one

* Add permissions in ListBlobsIncludeItems

* Add permissions in ListBlobsIncludeItems

* Appending the value to swagger

* Added Owner,Group,Permissions,Acl,ResourceType in ListBlob Response

* [Storage] Prep for stg 95 GA release (#23742)

* Prep for stg 95 GA release

* Update CHANGELOG.md

format for changelog

* Add SparseCheckoutPaths to archetype-typespec-emitter.yml (#23740)

Co-authored-by: Wei Hu <huwe@microsoft.com>

* [azopenai] Update azopenai changelog. (#23732)

* Fix missing stress matrix display name filter (#23726)

Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>

* update date (#23746)

* [azopenai] non-existing deployments fail in a different way now. (#23747)

* Increment package version after release of ai/azopenai (#23750)

* [Release] sdk/resourcemanager/hybridcompute/armhybridcompute/2.1.0-beta.1 generation from spec commit: 63d530d0def1c624f5d42d39170ff4ac196522e2 (#23753)

* Increment package version after release of security/keyvault/azadmin (#23749)

* Update azopenai* to use hardcoded variable group config (#23748)

* Update azopenai* to use hardcoded variable group config

* Remove bicep template for ai services

* [Release] sdk/resourcemanager/networkcloud/armnetworkcloud/1.2.0-beta.1 generation from spec commit: 4e46346c63935bace7e24f806b4b084f2589d660 (#23757)

* Add helpers to log a GitHub "notice" (#23764)

Co-authored-by: Mike Harder <mharder@microsoft.com>

* Increment package version after release of storage/azdatalake (#23745)

* Increment package version after release of storage/azblob (#23744)

* Increment package version after release of storage/azfile (#23743)

* batching adjustments for create-prjobmatrix (#23773)

Co-authored-by: Scott Beddall <scbedd@microsoft.com>

* changelog

* updated version

* Update sdk/ai/azopenaiassistants/CHANGELOG.md

Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com>

* updated

---------

Co-authored-by: ripark <ripark@microsoft.com>
Co-authored-by: Richard Park <51494936+richardpark-msft@users.noreply.github.com>
Co-authored-by: Azure SDK Bot <53356347+azure-sdk@users.noreply.github.com>
Co-authored-by: Daniel Jurek <djurek@microsoft.com>
Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com>
Co-authored-by: Chenjie Shi <tadelesh.shi@live.cn>
Co-authored-by: Ashley Stanton-Nurse <ashleyst@microsoft.com>
Co-authored-by: Scott Beddall <scbedd@microsoft.com>
Co-authored-by: Scott Beddall <45376673+scbedd@users.noreply.github.com>
Co-authored-by: Patrick Hallisey <hallipr@gmail.com>
Co-authored-by: ashruti-msft <137055338+ashruti-msft@users.noreply.github.com>
Co-authored-by: tanyasethi-msft <124860586+tanyasethi-msft@users.noreply.github.com>
Co-authored-by: Wei Hu <huwe@microsoft.com>
Co-authored-by: Grace Wilcox <43627800+gracewilcox@users.noreply.github.com>
Co-authored-by: Judy Liu <v-liujudy@microsoft.com>
Co-authored-by: Mike Harder <mharder@microsoft.com>
Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@richardpark-msft richardpark-msft richardpark-msft approved these changes

@jhendrixMSFT jhendrixMSFT Awaiting requested review from jhendrixMSFT jhendrixMSFT is a code owner

@chlowell chlowell Awaiting requested review from chlowell chlowell is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@glecaros @richardpark-msft