feat(modelarmor): Added snippets to sanitizer user prompt and model response
Added Model Armor samples for:
- Sanitize User Prompt
- Sanitize PDF file
- Sanitize Model Response
- Sanitize Model Response with User Prompt
Here is the summary of changes.
You are about to add 4 region tags.
- modelarmor/api/ModelArmor.Samples/SanitizeModelResponse.cs:17, tag
modelarmor_sanitize_model_response - modelarmor/api/ModelArmor.Samples/SanitizeModelResponseWithUserPrompt.cs:17, tag
modelarmor_sanitize_prompt_and_response - modelarmor/api/ModelArmor.Samples/SanitizeUserPrompt.cs:17, tag
modelarmor_sanitize_user_prompt - modelarmor/api/ModelArmor.Samples/ScanPdfFile.cs:17, tag
modelarmor_scan_pdf_file
This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:
- [ ] Refresh this comment
You need to solve the conflicts on the PR to be able to merge.
You need to solve the conflicts on the PR to be able to merge.
I have raised multiple PRs for Model Armor samples. Hence, I'll sequentially resolve merge conflicts and merge one-by-one.
I've resolved merge conflicts on this PR. Could you please run kokoro labels to trigger CI actions?
Tests are failing. These are API internal errors. How stable is the API? Even if we get it to green, are we certain we won't encounter service flakiness?
Tests are failing. These are API internal errors. How stable is the API? Even if we get it to green, are we certain we won't encounter service flakiness?
I just started facing this error today. Let me check with Model Armor team whether this is also a known issue
Were we able to get to a conclusion on the test failures here from the backend team?
Were we able to get to a conclusion on the test failures here from the backend team?
I discussed the issue with backend team yesterday. I will provide an update here accordingly.
Were we able to get to a conclusion on the test failures here from the backend team?
I discussed the issue with backend team yesterday. I will provide an update here accordingly.
@purva9413 Model Armor team confirmed that the issues have been resolved. Latest tests runs are successful.
@Anu4396 @purva9413 Thanks for the review! I've addressed all the feedback. Are we good to merge this PR?