-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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 llm_config in AgentOptimizer #2299
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2299 +/- ##
===========================================
+ Coverage 38.16% 50.03% +11.87%
===========================================
Files 78 78
Lines 7861 7868 +7
Branches 1681 1821 +140
===========================================
+ Hits 3000 3937 +937
+ Misses 4611 3600 -1011
- Partials 250 331 +81
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Thanks for addressing this issue. However, where can I find information on using this revised code. I see references to llm_config and ao_config_list. Thanks. |
* support llm_config in agentoptimizer * fix doc * restore seed timeout --------- Co-authored-by: “skzhang1” <“[email protected]”> Co-authored-by: Chi Wang <[email protected]>
Why are these changes needed?
AgentOptimizer with OAI_CONFIG_LIST in its constructor is forcing users to learn the LLM config utilities.
Taking llm_config as inout is more convenient for users.
Related issue number
Closes #2279
Checks