Skip to content

feat(vectorizer): add configure.vectors.multi2VecAWS() module config builder - #479

Open
OsamaAnsar wants to merge 2 commits into
weaviate:mainfrom
OsamaAnsar:feature/multi2vec-aws
Open

OsamaAnsar wants to merge 2 commits into
weaviate:mainfrom
OsamaAnsar:feature/multi2vec-aws

Conversation

@OsamaAnsar

Copy link
Copy Markdown
Contributor

Adds a configure.vectors.multi2VecAWS() builder for the multi2vec-aws (Bedrock multimodal embeddings) vectorizer, bringing the TypeScript client to parity with the Python client, which already ships multi2vec-aws in config_vectorizers.py.

The Multi2VecAWSConfig return type and Multi2VecAWSConfigCreate options type already existed in the client — this wires up the missing pieces:

  • 'multi2vec-aws' added to the Vectorizer union and to VectorizerConfigType / VectorizerConfigCreateType
  • configure.vectors.multi2VecAWS() builder, mirroring multi2VecNvidia / multi2VecCohere — supports region, model, dimensions, and imageFields / textFields with optional per-field weights
  • 3 unit tests: defaults (→ config: undefined), all values, values + weights

Follow-up to #345, which tracked both multi2vec-aws and text2vec-morph — only text2vec-morph landed on the TS side.

npm run test:unit and tsc --noEmit pass; prettier / eslint clean.

@orca-security-eu orca-security-eu Bot left a comment •

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Infrastructure as Code high 0   medium 0   low 0   info 0 View in Orca
Passed Passed SAST high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Vulnerabilities high 0   medium 0   low 0   info 0 View in Orca

@weaviate-git-bot

Copy link
Copy Markdown

Great to see you again! Thanks for the contribution.

beep boop - the Weaviate bot 👋🤖

PS:
Are you already a member of the Weaviate Forum?

This branch has not been deployed

No deployments
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