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

feat: Add blog draft for the Sparse Vectors based RecSys #27

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

NirantK
Copy link
Contributor

@NirantK NirantK commented Feb 3, 2024

No description provided.

})

for row in ratings.itertuples():
user_sparse_vectors[row.user_id]["values"].append(row.rating)
Copy link
Member

Choose a reason for hiding this comment

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

compute user_sparse_vectors[row.user_id] only once and bind it to a variable :)

@mjang mjang self-requested a review February 5, 2024 15:33
@mjang
Copy link
Contributor

mjang commented Feb 21, 2024

@NirantK , as I read through your work, I realized this sounded similar to https://qdrant.tech/documentation/tutorials/search-beginners/

IOW, I'm tempted to set this up as a new doc page, something like "Semantic search for advanced users". (Does that make sense? Does this include more advanced techniques?)

@NirantK
Copy link
Contributor Author

NirantK commented Feb 22, 2024

@mjang This is not a beginner-friendly article or blog at all. This is a novel and experimental way to model recommendation systems in Qdrant from @generall. I'm inclined to believe that this is best treated as either an article or a blog?

@mjang
Copy link
Contributor

mjang commented Feb 22, 2024

@mjang This is not a beginner-friendly article or blog at all. This is a novel and experimental way to model recommendation systems in Qdrant from @generall. I'm inclined to believe that this is best treated as either an article or a blog?

Thanks for explaining, @NirantK . We'll proceed with this as an article, see what the reaction is. If it's positive, I might revisit the idea.

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.

3 participants