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

Support structured output #3732

Merged
merged 11 commits into from
Oct 13, 2024
Merged

Support structured output #3732

merged 11 commits into from
Oct 13, 2024

Conversation

lordlinus
Copy link
Collaborator

Why are these changes needed?

GPT-4o-2024-08-06 supports structured output with new beta API and this PR will use that API as part of extra_args to return the structured output format. remove stream option since it is not supported. Includes sample cookbook to show usage

Related issue number

Checks

@lordlinus lordlinus marked this pull request as ready for review October 10, 2024 08:56
@lordlinus
Copy link
Collaborator Author

@jackgerrits and @ekzhu PR ready for structured output support. Let me know your thoughts

@rysweet rysweet added the awaiting-op-response Issue or pr has been triaged or responded to and is now awaiting a reply from the original poster label Oct 10, 2024
@lordlinus lordlinus marked this pull request as draft October 11, 2024 01:42
@lordlinus lordlinus marked this pull request as ready for review October 13, 2024 03:57
@lordlinus
Copy link
Collaborator Author

@ekzhu can you please review now. please note the failing test is not related to this PR.

@ekzhu ekzhu changed the base branch from staging to main October 13, 2024 21:26
@ekzhu ekzhu merged commit a106229 into microsoft:main Oct 13, 2024
30 of 31 checks passed
rysweet added a commit that referenced this pull request Oct 14, 2024
* CodeQL advanced config (#3736)

* CodeQL advanced config with dotnet build

* Update codeql.yml

---------

Co-authored-by: Ryan Sweet <[email protected]>

* The /python/benchmarks folder simply contained a note indicating that the benchmarks were moved. This commit deletes this note and directory. (#3735)

* Update quick start examples to illustrate how to set up model client completely (#3739)

* Update FunctionCallGenerator.cs to address race condition (#3758)

Update FunctionCallGenerator.cs to address file name race condition with simultaneous builds

* Skip Bing tests when no API key is present. (#3757)

* Skip Bing pytests when no API key is present.

* Fixed formatting.

---------

Co-authored-by: Ryan Sweet <[email protected]>

* Move docker code exec to autogen-ext (#3733)

* move docker code exec to autogen-ext

* fix test

* rename docker subpackage

* add missing renamed package

---------

Co-authored-by: Leonardo Pinheiro <[email protected]>

* Add initial extensions doc page (#3762)

* Update README.md (#3768)

* Adding Benchmarks back into agbench and updates to agbench (#3711)

* Correcting Typo in README.md (#3770)

* Make sure exceptions in process publish is logged. (#3774)

* Support structured output (#3732)

* Support structured output

* use ruff format

* add type checking for cookbook

* add the notebook to index.md

* fix the type error

* pass response_format explicitly

* remove casting

* ensure type are correct

* seperate response_format arg

* fix type and resolve pyright errors

---------

Co-authored-by: Eric Zhu <[email protected]>

* Update README.md (#3777)

* Update indexes for better navigation (#3779)

* Update indexes for better navigation

* Fix link

* Fix link

* Create a notebook to demonstrate handoff pattern (#3778)

* Add work in progress message to agentchat on home (#3784)

* Fill spelling mistake (#3786)

* Update README.md to clarify v2 vs v4 (#3785)

Co-authored-by: gagb <[email protected]>

---------

Co-authored-by: Jack Gerrits <[email protected]>
Co-authored-by: afourney <[email protected]>
Co-authored-by: Eric Zhu <[email protected]>
Co-authored-by: Max Golovanov <[email protected]>
Co-authored-by: Leonardo Pinheiro <[email protected]>
Co-authored-by: Leonardo Pinheiro <[email protected]>
Co-authored-by: Hussein Mozannar <[email protected]>
Co-authored-by: vikas434 <[email protected]>
Co-authored-by: Sunil Sattiraju <[email protected]>
Co-authored-by: gagb <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting-op-response Issue or pr has been triaged or responded to and is now awaiting a reply from the original poster
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants