Skip to content

feat: add GpuIndexHNSW — GPU HNSW search with OCQ beam search kernel (slim, feature-only)#3

Closed
devin-ai-integration[bot] wants to merge 1 commit into
mainfrom
devin/gpu-hnsw-feature-only
Closed

feat: add GpuIndexHNSW — GPU HNSW search with OCQ beam search kernel (slim, feature-only)#3
devin-ai-integration[bot] wants to merge 1 commit into
mainfrom
devin/gpu-hnsw-feature-only

Conversation

@devin-ai-integration

Copy link
Copy Markdown

Summary

Slim, review-friendly version of the GPU HNSW feature currently on gpu-hnsw-faiss (PR #1).

The gpu-hnsw-faiss branch mixes the feature with a full upstream faiss OSS sync (via a Merge upstream/main commit), so its diff-vs-main is ~9.5k lines / 194 files. This branch rebases only the feature onto current main and squashes the noisy development history (perf experiments + their reverts) into one commit, so the diff is exactly the feature:

faiss/gpu/GpuIndexHNSW.{cu,h}
faiss/gpu/impl/GpuHnsw{Build,Search,SearchKernel}.cuh
faiss/gpu/impl/GpuHnswTypes.{cu,h}
faiss/gpu/CMakeLists.txt
→ 8 files, +1898

No upstream sync, no reverted char4/dp4a experiments — just the final feature.

Relationship to other PRs:

Not build-verified locally (no CUDA toolchain wired for a full build in this session); tree is byte-identical to gpu-hnsw-faiss^1.

Link to Devin session: https://6sense.devinenterprise.com/sessions/d39aba56b8a3467cbbf231ab631a06ed

Feature-only, rebased onto current main. Squashes the GPU HNSW development
history (perf experiments + their reverts) into a single commit and drops
the unrelated upstream faiss sync that the gpu-hnsw-faiss branch also carried,
so the diff is just the 8 GPU HNSW files (+1898).

Signed-off-by: Devin AI <devin-ai-integration[bot]@users.noreply.github.com>
@devin-ai-integration

Copy link
Copy Markdown
Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@devin-ai-integration

Copy link
Copy Markdown
Author

Superseded: this content is now folded directly into the clean gpu-hnsw-faiss branch (rebased onto synced upstream master, dead-code/GPU_TIMING cleanups included). Closing.

@devin-ai-integration
devin-ai-integration Bot deleted the devin/gpu-hnsw-feature-only branch July 9, 2026 06:31
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.

0 participants