- 
                Notifications
    You must be signed in to change notification settings 
- Fork 486
misc: Update policy-guided safeguard to support initiate_group_chat API #2121
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
misc: Update policy-guided safeguard to support initiate_group_chat API #2121
Conversation
… to add additional hooks and include logic to enable safeguard policy format and tool message bug handling add changes for type check workflow fix "Argument "target" to "LLMGuardrail" has incompatible type "None"; expected "TransitionTarget" " fix type cheking issues & better formatting fix type error in test file
update safeguards doc
…tor is running all tools)
…h/ag2_maris into maris_groupchat_initiate
| Look good in general. @jiancui-research I have a few questions. If yes, could you add an example in the notebook and update the documentation? Could you fix the pre-commit error as well? | 
…h/ag2_maris into maris_groupchat_initiate
fix pre commit error
| @randombet, thanks for your comment. I double checked. The current implementation also supports  | 
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Why are these changes needed?
Update to support new API:
initiate_group_chat. Previously, to use the policy-guided safeguard with initiate_chat it would be:To use with
initiate_group_chat:This PR updates the code & documentation to reflect the new recommended approach
Related issue number
Checks