Skip to content

fix: add qdrant support to vm setup and vector benchmark defaults#147

Merged
kacy merged 1 commit intomainfrom
bench/refresh-vector-defaults
Feb 15, 2026
Merged

fix: add qdrant support to vm setup and vector benchmark defaults#147
kacy merged 1 commit intomainfrom
bench/refresh-vector-defaults

Conversation

@kacy
Copy link
Copy Markdown
Owner

@kacy kacy commented Feb 15, 2026

summary

  • setup-vm-vector.sh now pulls the qdrant docker image and installs qdrant-client pip package
  • bench-vector.sh defaults QDRANT=true for full comparison runs (previously off by default)
  • added --no-qdrant flag for skipping qdrant when docker isn't available
  • updated SIFT1M note in bench/README.md — the dataset fits comfortably in 32GB (c2-standard-8), not the previously stated c2-standard-16
  • added qdrant column to vector comparison tables in bench/README.md

what was tested

  • verified bench-vector.sh --help-style flag parsing accepts both --qdrant and --no-qdrant
  • confirmed setup script additions are syntactically correct
  • reviewed README table formatting

qdrant was added as a benchmark target but wasn't included in the
vm setup script and defaulted to off in bench-vector.sh. now:

- setup-vm-vector.sh pulls qdrant docker image and installs qdrant-client
- bench-vector.sh defaults QDRANT=true for full runs
- added --no-qdrant flag for environments without docker
- updated SIFT1M note — fits in 32GB (c2-standard-8), not 16+ as stated
- added qdrant column to vector comparison tables in README
@kacy kacy merged commit db975c2 into main Feb 15, 2026
7 checks passed
@kacy kacy deleted the bench/refresh-vector-defaults branch February 15, 2026 13:10
kacy added a commit that referenced this pull request Feb 19, 2026
qdrant was added as a benchmark target but wasn't included in the
vm setup script and defaulted to off in bench-vector.sh. now:

- setup-vm-vector.sh pulls qdrant docker image and installs qdrant-client
- bench-vector.sh defaults QDRANT=true for full runs
- added --no-qdrant flag for environments without docker
- updated SIFT1M note — fits in 32GB (c2-standard-8), not 16+ as stated
- added qdrant column to vector comparison tables in README
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