-
Notifications
You must be signed in to change notification settings - Fork 380
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
Initial work on support for exceptions. #384
Conversation
5313211
to
5e79b60
Compare
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.
Looks good so far! 👍
a2f82fa
to
cb88837
Compare
related issue #634 |
@Pouyanpi : can you sort the conflicts and add a short documentation to the configuration guide. Thanks! |
… for exceptions by rdinu)
…ation, and llama guard flows with allow_exceptions
Rename exception classes in cofiles
cb88837
to
7ee2fb9
Compare
@drazvan, I made the changes, please have a look. |
cc @Pouyanpi: this would break QA tests and demo flows with the ABC bot.
Thanks @Pouyanpi. I had to fix a bug related to the stop behavior, see my commit, and tweak a bit more the docs. We need to add tests for the rest, but we'll do in a separate PR. |
WIP. Still needs: