Skip to content

Conversation

bevzzz
Copy link
Collaborator

@bevzzz bevzzz commented May 26, 2025

Unit tests to make sure vectors in WeaviateObject are serialized correctly regardless of how they are combined.
Most importantly we want to make sure single- and multi-vectors can be combined under "vectors".

This change is a cosmetic change to make sure the output JSON does not contain empty "vector": {} kv-pair.

@bevzzz bevzzz self-assigned this May 26, 2025
@bevzzz bevzzz requested a review from antas-marcin May 26, 2025 13:23
Copy link

@orca-security-eu orca-security-eu bot left a comment

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

@bevzzz bevzzz marked this pull request as ready for review May 26, 2025 13:48
Multi-vectors can only be created as named vectors.
@bevzzz bevzzz merged commit 03cf63d into main Jun 3, 2025
5 checks passed
@bevzzz bevzzz deleted the test/multi-vector-adapter branch June 3, 2025 08:44
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.

1 participant