You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Is your feature request related to a problem? Please describe.
Right now, config_list, llm_configs and code_execution_config are dictionaries, and the schema of these dictionaries are enforced through parser functions.
Describe the solution you'd like
A better way to handle this would be to use Pydantic models so parsing, serializing and data validations are handled by Pydantic. This also address many issues related to invalid configs.
Arguably I would add code_execution_config to that as well.
ekzhu
changed the title
[Feature Request]: Use Pydantic models for llm_configs and config_list
[Feature Request]: Use Pydantic models for llm_configs, config_list and code_execution_configJan 2, 2024
Is your feature request related to a problem? Please describe.
Right now,
config_list
,llm_configs
andcode_execution_config
are dictionaries, and the schema of these dictionaries are enforced through parser functions.Describe the solution you'd like
A better way to handle this would be to use Pydantic models so parsing, serializing and data validations are handled by Pydantic. This also address many issues related to invalid configs.
Additional context
Pydantic
Example of Pydantic models already used in our repo: https://github.com/microsoft/autogen/blob/main/autogen/function_utils.py
Existing issues to address data validation:
#1082
#1026
#1074
#1069
The text was updated successfully, but these errors were encountered: