Skip to content

Update __init__.py#2628

Closed
asadoughi wants to merge 1 commit intometa-pytorch:mainfrom
asadoughi:export-D67118292
Closed

Update __init__.py#2628
asadoughi wants to merge 1 commit intometa-pytorch:mainfrom
asadoughi:export-D67118292

Conversation

@asadoughi
Copy link

Summary:
Fix erroneous logging Failed to load GPU Faiss: No module named 'swigfaiss_gpu'. Will not load constructor refs for GPU indexes. when I was running on a GPU-based index just fine.

GpuIndex... is loaded via loader.py.

X-link: facebookresearch/faiss#4086

Reviewed By: mnorris11

Differential Revision: D67118292

Pulled By: asadoughi

Summary:
Fix erroneous logging `Failed to load GPU Faiss: No module named 'swigfaiss_gpu'. Will not load constructor refs for GPU indexes.` when I was running on a GPU-based index just fine.

`GpuIndex...` is loaded via loader.py.

X-link: facebookresearch/faiss#4086

Reviewed By: mnorris11

Differential Revision: D67118292

Pulled By: asadoughi
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Dec 12, 2024
@facebook-github-bot
Copy link
Contributor

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

facebook-github-bot pushed a commit to facebookresearch/faiss that referenced this pull request Dec 12, 2024
Summary:
X-link: meta-pytorch/torchrec#2628

Fix erroneous logging `Failed to load GPU Faiss: No module named 'swigfaiss_gpu'. Will not load constructor refs for GPU indexes.` when I was running on a GPU-based index just fine.

`GpuIndex...` is loaded via loader.py.

Pull Request resolved: #4086

Test Plan:
before
```
(faiss_gpu_cuvs_nightly) [sadoughi@devvm15198.frc0 ~]$ python3 script.py
Failed to load GPU Faiss: No module named 'swigfaiss_gpu'. Will not load constructor refs for GPU indexes.
```

after
```
(faiss_gpu_cuvs_nightly) [sadoughi@devvm15198.frc0 ~]$ python3
Python 3.12.8 | packaged by conda-forge | (main, Dec  5 2024, 14:24:40) [GCC 13.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import faiss
>>>
```

Reviewed By: mnorris11

Differential Revision: D67118292

Pulled By: asadoughi

fbshipit-source-id: aa05aa20fe61e4ad75d4234f52e3b0d6a673a0b9
samanthawaters8882michaeldonovan added a commit to samanthawaters8882michaeldonovan/faiss that referenced this pull request Oct 12, 2025
Summary:
X-link: meta-pytorch/torchrec#2628

Fix erroneous logging `Failed to load GPU Faiss: No module named 'swigfaiss_gpu'. Will not load constructor refs for GPU indexes.` when I was running on a GPU-based index just fine.

`GpuIndex...` is loaded via loader.py.

Pull Request resolved: facebookresearch/faiss#4086

Test Plan:
before
```
(faiss_gpu_cuvs_nightly) [sadoughi@devvm15198.frc0 ~]$ python3 script.py
Failed to load GPU Faiss: No module named 'swigfaiss_gpu'. Will not load constructor refs for GPU indexes.
```

after
```
(faiss_gpu_cuvs_nightly) [sadoughi@devvm15198.frc0 ~]$ python3
Python 3.12.8 | packaged by conda-forge | (main, Dec  5 2024, 14:24:40) [GCC 13.3.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import faiss
>>>
```

Reviewed By: mnorris11

Differential Revision: D67118292

Pulled By: asadoughi

fbshipit-source-id: aa05aa20fe61e4ad75d4234f52e3b0d6a673a0b9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants