Skip to content

ci: adding FAISS github workflow#2879

Merged
davidsbatista merged 21 commits intomainfrom
feat/adding-workflow-for-FAISS
Feb 27, 2026
Merged

ci: adding FAISS github workflow#2879
davidsbatista merged 21 commits intomainfrom
feat/adding-workflow-for-FAISS

Conversation

@davidsbatista
Copy link
Copy Markdown
Contributor

Checklist

@github-actions github-actions Bot added the type:documentation Improvements or additions to documentation label Feb 26, 2026
@davidsbatista davidsbatista marked this pull request as ready for review February 27, 2026 11:47
@davidsbatista davidsbatista requested a review from a team as a code owner February 27, 2026 11:47
@davidsbatista davidsbatista requested review from julian-risch and removed request for a team February 27, 2026 11:47
Comment thread integrations/faiss/pyproject.toml
@julian-risch julian-risch self-requested a review February 27, 2026 12:30
Copy link
Copy Markdown
Member

@julian-risch julian-risch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's add :raises: to docstrings. Otherwise looks good to me!


:param documents: The list of documents to write.
:param policy: The policy to handle duplicate documents.
:return: The number of documents written.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:raises: 
    ValueError: If ...
    DuplicateDocumentError: If ...
    DocumentStoreError: If ...

:param query_embedding: The query embedding.
:param top_k: The number of results to return.
:param filters: Filters to apply.
:return: A list of matching Documents.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:raises:
    FilterError: If ...

@julian-risch julian-risch changed the title feat: adding FAISS github workflow ci: adding FAISS github workflow Feb 27, 2026
Copy link
Copy Markdown
Member

@julian-risch julian-risch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 👍

@davidsbatista davidsbatista merged commit 99f63fd into main Feb 27, 2026
8 checks passed
@davidsbatista davidsbatista deleted the feat/adding-workflow-for-FAISS branch February 27, 2026 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

integration:faiss topic:CI type:documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants