-
Notifications
You must be signed in to change notification settings - Fork 5.1k
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
Add support for changing system message role - this allows using o1-preview and o1-mini models #3591
base: 0.2
Are you sure you want to change the base?
Conversation
@microsoft-github-policy-service agree |
I see https://github.com/microsoft/autogen/pull/3530/files just now - happy to close this one if not needed |
Thanks for the PR, we're still working on O1 support. Did you want to try out that other PR or provide feedback? |
This PR is against AutoGen 0.2. AutoGen 0.2 has been moved to the 0.2 branch. Please rebase your PR on the 0.2 branch or update it to work with the new AutoGen 0.4 that is now in main. |
Why are these changes needed?
To use the o1-preview and o1-mini models, we cannot send messages with role "system", to deal with this, I allowed controlling the role of the system message via an optional param.
Related issue number
N/A
Checks