Skip to content
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 amazon-bedrock-guardrailConfig #59

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

supreetkt
Copy link

@supreetkt supreetkt commented May 24, 2024

Add support for amazon-bedrock-guardrailConfig

  • Adds guardrailConfig to hold values for this field within the guardrails property for Bedrock classes and this is passed to the bedrock invoke model calls via the amazon-bedrock-guardrailConfig field in the body. amazon-bedrock-guardrailConfig allows adding a tagSuffix [documentation] and different streaming response behaviors [documentation]
  • Also cleaned up some documentation, error messages and tests
  • Added a conftest file for common fixtures that can be used across tests

Closes: #52

@supreetkt supreetkt marked this pull request as ready for review May 24, 2024 22:12
@supreetkt
Copy link
Author

@3coins: can we trigger the workflow checks on this PR? Thanks.

@nbaillie
Copy link

is there any idea when this might be released?

@supreetkt
Copy link
Author

is there any idea when this might be released?

I haven't heard back from @3coins yet, but hopefully soon!

@paulonasc
Copy link

@3coins any updates on this PR? I need this feature ASAP. Please let me know if I can help :)

@3coins
Copy link
Collaborator

3coins commented Jun 24, 2024

@supreetkt
Can you fix conflicts and rebase from main.

@fpSpindola
Copy link

hey guys! Is there any chance this is going to be released soon?

@john-mihailescu

This comment was marked as outdated.

@KimYongKuk
Copy link

I really hope this gets implemented soon.

@supreetkt
Copy link
Author

@supreetkt Can you fix conflicts and rebase from main.

yes, I can resolve the conflicts. Missed notifications on this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Adding amazon-bedrock-guardrailConfig for Guardrails to support different streaming types
7 participants