Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

FAISS load to None #6887

Open
brainer3220 opened this issue May 9, 2024 · 1 comment
Open

FAISS load to None #6887

brainer3220 opened this issue May 9, 2024 · 1 comment

Comments

@brainer3220
Copy link

Describe the bug

I've use FAISS with Datasets and save to FAISS.

Then load to save FAISS then no error, then ds to None

ds.load_faiss_index('embeddings', 'my_index.faiss')

Steps to reproduce the bug

1.

ds_with_embeddings = ds.map(lambda example: {'embeddings': model(transforms(example['image']).unsqueeze(0)).squeeze()}, batch_size=64)

ds_with_embeddings.add_faiss_index(column='embeddings')

ds_with_embeddings.save_faiss_index('embeddings', 'index.faiss')

2.

ds.load_faiss_index('embeddings', 'my_index.faiss')

Expected behavior

Add column in Datasets.

Environment info

Google Colab, SageMaker Notebook

@Dref360
Copy link
Contributor

Dref360 commented May 16, 2024

Hello,

I'm not sure I understand.
The return value of ds.load_faiss_index is None as expected.

I see that loading an Index on a dataset that doesn't have an embedding column doesn't raise an Issue. Is that the issue?

So ds doesn't have an embedding column, but we load an index that looks for it. But this will raise an issue only when calling ds.search.

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

No branches or pull requests

2 participants