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

Retriever base type #16

Merged
merged 2 commits into from
Apr 23, 2024

Conversation

willtai
Copy link
Contributor

@willtai willtai commented Apr 22, 2024

Summary

  • Create Retriever base type. This makes more sense semantically and the other retriever methods can inherit from this class.
  • Adds comment in docstring to pass correct Embedder type

@willtai willtai requested a review from oskarhane April 22, 2024 15:02
@willtai willtai force-pushed the willtai/retriever-base-type-and-embedder-error-handling branch 2 times, most recently from 7b5ac29 to 792bfb6 Compare April 22, 2024 15:06
@willtai willtai force-pushed the willtai/retriever-base-type-and-embedder-error-handling branch from 792bfb6 to e6b43dd Compare April 22, 2024 15:08
@willtai willtai marked this pull request as ready for review April 22, 2024 15:16
Copy link
Member

@oskarhane oskarhane left a comment

Choose a reason for hiding this comment

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

I think this makes sense 👍

@willtai willtai merged commit 944c4e9 into main Apr 23, 2024
6 checks passed
@willtai willtai deleted the willtai/retriever-base-type-and-embedder-error-handling branch April 23, 2024 08:30
willtai added a commit that referenced this pull request Apr 23, 2024
* Refactored Retriever object

* Added comment about Embedder type in docstring
willtai added a commit that referenced this pull request Apr 23, 2024
* Refactored Retriever object

* Added comment about Embedder type in docstring
willtai added a commit that referenced this pull request Apr 23, 2024
* Refactored Retriever object

* Added comment about Embedder type in docstring
willtai added a commit that referenced this pull request Apr 23, 2024
* Refactored Retriever object

* Added comment about Embedder type in docstring
willtai added a commit that referenced this pull request Apr 23, 2024
* Adds Getting Started instructions to README

* Retriever base type (#16)

* Refactored Retriever object

* Added comment about Embedder type in docstring

* Adds Getting Started instructions and CLA Check workflow
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.

2 participants