Skip to content

Conversation

holyMolyTolli
Copy link
Contributor

No description provided.

@holyMolyTolli holyMolyTolli self-assigned this Oct 6, 2025
Copy link

netlify bot commented Oct 6, 2025

Deploy Preview for condescending-goldwasser-91acf0 ready!

Name Link
🔨 Latest commit 0b694d2
🔍 Latest deploy log https://app.netlify.com/projects/condescending-goldwasser-91acf0/deploys/68f38a3a7af89900088dbeff
😎 Deploy Preview https://deploy-preview-1930--condescending-goldwasser-91acf0.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

"""Semantic chunking: uses embedding similarity to find natural breaks"""
from llama_index.core import Document

semantic_splitter = SemanticSplitterNodeParser(
Copy link
Contributor

Choose a reason for hiding this comment

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

Sabrina says in the video that if one wants to learn more about the parameters of this one, we'll provide links below
There're no links afaik
Maybe we should add something?

…ayload indexes for filtering and grouping to pitstop project.
@holyMolyTolli holyMolyTolli mentioned this pull request Oct 7, 2025

```python
# Group by movie name to get unique recommendations
response = client.query_points_groups(
Copy link
Contributor

Choose a reason for hiding this comment

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

Revisiting my comment on "groups" won't even work without an index in other deployments
So I assume it's connected to the strict_mode config
So most probably if it's disabled (or like here, :memory: mode, where we don't have HNSW at all), it still will work without payload_index, just ineffectively (full scans)
Which is ig fine for this lesson, but I'd still keep warning people that the moment they switch to production, payload indices on group fields should be always set up and before building HNSW, if filterable HNSW is needed

Copy link
Contributor Author

Choose a reason for hiding this comment

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

So most probably if it's disabled (or like here, :memory: mode, where we don't have HNSW at all), it still will work without payload_index, just ineffectively (full scans)

I am not sure about the claim that we don't have HNSW when we use :memory: mode. My understanding is that HNSW index is still build but ephemeral and lost on process exit.

Copy link
Contributor Author

@holyMolyTolli holyMolyTolli Oct 10, 2025

Choose a reason for hiding this comment

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

From the client readme

client = QdrantClient(":memory:") # or QdrantClient(path="path/to/db") for local mode and persistent storage

So the parameter for QdrantClient just defines where Qdrant runs and where data is stored, not if HNSW is build:
QdrantClient(":memory:") --> Ephemeral. No Qdrant server.
QdrantClient(path="path/to/db") --> Persistent storage. No Qdrant server.
QdrantClient(url="http://localhost:6333") --> Persistent in server’s storage path.
QdrantClient(url="https://xxx.cloud.qdrant.io:6333", api_key="...") --> Persistent in cloud.

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