Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Throw when attempting to move IndexPQ to GPU #3328

Closed
wants to merge 1 commit into from

Conversation

ramilbakhshyiev
Copy link
Contributor

Differential Revision: D55476917

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D55476917

Summary: Pull Request resolved: #3328

Differential Revision: D55476917
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D55476917

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 77e2e79.

@Sheshansh
Copy link

Sheshansh commented Mar 30, 2024

@ramilbakhshyiev incidentally I was just looking to run IndexPQ on GPU.
My code was too slow and not using GPUs and this PR suggests it doesn't exist.

Is anyone working on IndexPQ for GPU? If not, I can do this.

@ramilbakhshyiev ramilbakhshyiev deleted the export-D55476917 branch April 1, 2024 22:29
@ramilbakhshyiev
Copy link
Contributor Author

Hey @Sheshansh, no one is currently working on it. Please go ahead and thank you, appreciate your time!

@algoriddle
Copy link
Contributor

@Sheshansh, you can try IVF with nlist=1 as a workaround (or proof-of-concept, if you like).

@Sheshansh
Copy link

Oh, thanks @algoriddle I didn't know IVFPQ is available on GPU. This simplifies things. I would test it first.

abhinavdangeti pushed a commit to blevesearch/faiss that referenced this pull request Jul 12, 2024
Summary: Pull Request resolved: facebookresearch#3328

Reviewed By: junjieqi

Differential Revision: D55476917

fbshipit-source-id: e7f64adefa07650fda32ad2300a1b933cedc9c79
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants