Skip to content

Add an option to set multimodal storage destination #979

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

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

okamoto-aws
Copy link
Collaborator

Description of Changes

Please explain the changes in detail.
If there is any impact on existing users (compatibility, degradation, breaking changes, etc.), be sure to include it in the explanation.

  • Add ragKnowledgeBaseMultiModalStorage to enable parsing image files (png/jpeg) directly
  • Update documentation

Note that enabling the option requires remake of the existing knowledge base.

Checklist

  • Executed npm run lint
  • Modified relevant documentation
  • Verified operation in local environment
  • Executed npm run cdk:test and if there are snapshot differences, execute npm run cdk:test:update-snapshot to update snapshots

Related Issues

#895

Copy link
Contributor

@tbrand tbrand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You also have to commit the cdk.json changes.

@@ -98,6 +98,7 @@ export const stackInputSchema = z.object({
embeddingModelId: z.string().default('amazon.titan-embed-text-v2:0'),
ragKnowledgeBaseStandbyReplicas: z.boolean().default(false),
ragKnowledgeBaseAdvancedParsing: z.boolean().default(false),
ragKnowledgeBaseMultiModalStorage: z.boolean().default(true),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought the default value is false. Is this correct?

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.

2 participants