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

Add new example: Build a search engine quickly using Qdrant #30

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

geetu040
Copy link

This pull request adds a new example demonstrating the integration of Qdrant Search Engine into various applications. The example provides a step-by-step guide for building a search engine very quickly using Qdrant. It will help out the developers that are new to qdrant and are looking to get started quickly. It uses Quora Question Pair dataset to quickly build a search engine. A detailed guide to the code in a medium article is also linked in the code. There is no existing example in the repository that is using qdrant this straight-forward so it makes sense to add an example that shows simplest and easiest way to create a practical search engine. Here are the links to the work that has been done in this example
Medium Article: https://medium.com/@raoarmaghanshakir040/build-a-search-engine-in-5-minutes-using-qdrant-f43df4fbe8d1
Kaggle Code: https://www.kaggle.com/code/sacrum/quora-search-engine-using-qdrant

Copy link
Contributor

@NirantK NirantK left a comment

Choose a reason for hiding this comment

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

Is the notebook and Python script logically identical? If yes, we should only retain one of those.

Copy link
Contributor

@NirantK NirantK left a comment

Choose a reason for hiding this comment

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

Is the notebook and Python script logically identical? If yes, we should only retain one of those.

quick-search-engine/README.md Outdated Show resolved Hide resolved
quick-search-engine/README.md Outdated Show resolved Hide resolved
quick-search-engine/README.md Outdated Show resolved Hide resolved
@geetu040
Copy link
Author

geetu040 commented Mar 4, 2024

Understood, I'll make the changes

Is the notebook and Python script logically identical? If yes, we should only retain one of those.

Yes, I'll remove the Python script

quick-search-engine/README.md Outdated Show resolved Hide resolved
quick-search-engine/requirements.txt Outdated Show resolved Hide resolved
quick-search-engine/README.md Outdated Show resolved Hide resolved
Copy link
Contributor

@NirantK NirantK left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@NirantK
Copy link
Contributor

NirantK commented Mar 28, 2024

Request your review @davidmyriel since I'm not sure if there are specific guidelines or requirements for adding an example. I've checked the notebook, README, code organisation for hygiene and skimmed enough of the notebook to believe that it's correct.

@geetu040
Copy link
Author

Hi @davidmyriel, can you please review this PR?

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