-
-
Notifications
You must be signed in to change notification settings - Fork 4.7k
Litellm noma guardrail support images #16199
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
Litellm noma guardrail support images #16199
Conversation
|
@dean-zavad is attempting to deploy a commit to the CLERKIEAI Team on Vercel. A member of the Team first needs to authorize it. |
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.
@dean-zavad can you add apply_guardrail support in this PR - it would allow noma to work across all LLM endpoints (/v1/responses, /v1/messages, /audio/transcription, etc.)
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.
we will implement that in another PR. apply_guardrail broke the current implementation. @ishaan-jaff
Guardrail Image Support
Relevant issues
Pre-Submission checklist
Please complete all items before asking a LiteLLM maintainer to review your PR
tests/litellm/directory, Adding at least 1 test is a hard requirement - see detailsmake test-unitType
🆕 New Feature
🐛 Bug Fix
Changes
Notes
We removed apply_guardrails hook from noma code from several reasons:
Once the above issues will be resolved, we'll be happy to opt in
Screenshot Of Passing Tests