Skip to content

fix: allow update on HNSWLibIndex#1604

Merged
JoanFM merged 4 commits into
mainfrom
fix-update-hnswlib-index
Jun 1, 2023
Merged

fix: allow update on HNSWLibIndex#1604
JoanFM merged 4 commits into
mainfrom
fix-update-hnswlib-index

Conversation

@JoanFM

@JoanFM JoanFM commented May 31, 2023

Copy link
Copy Markdown
Member

No description provided.

Signed-off-by: Joan Fontanals Martinez <joan.martinez@jina.ai>
@JoanFM JoanFM force-pushed the fix-update-hnswlib-index branch from bf49d1c to 86cc22a Compare May 31, 2023 16:34
@JoanFM JoanFM linked an issue May 31, 2023 that may be closed by this pull request
@JoanFM JoanFM changed the title fix: allow update on hnswlib fix: allow update on HNSWLibIndex May 31, 2023
Comment thread docs/user_guide/storing/index_hnswlib.md Outdated

@samsja samsja left a comment

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.

Is it possible to have a code snippet in docs ?

Signed-off-by: Joan Fontanals Martinez <joan.martinez@jina.ai>
@JoanFM JoanFM requested a review from samsja June 1, 2023 07:19
@JoanFM

JoanFM commented Jun 1, 2023

Copy link
Copy Markdown
Member Author

Is it possible to have a code snippet in docs ?

Can u recheck?

```

Check [here](docindex#nested-data-with-subindex) for nested data with subindex. No newline at end of file
Check [here](docindex#nested-data-with-subindex) for nested data with subindex.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this link doesn't work let's remove it

I'll add subindex feature info in a separate pr

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

check now

text: str
embedding: NdArray[128]

docs = DocList[MyDoc](

@jupyterjazz jupyterjazz Jun 1, 2023

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'd show an example where we retrieve some data (find, filter..) and replace only those docs, otherwise I don't think this is a relatable example

Signed-off-by: Joan Fontanals Martinez <joan.martinez@jina.ai>
@JoanFM JoanFM requested a review from jupyterjazz June 1, 2023 13:36
Comment thread docs/user_guide/storing/index_hnswlib.md
@JoanFM JoanFM requested review from jupyterjazz and samsja June 1, 2023 14:15
@github-actions github-actions Bot added size/m and removed size/s labels Jun 1, 2023
@github-actions

github-actions Bot commented Jun 1, 2023

Copy link
Copy Markdown

📝 Docs are deployed on https://ft-fix-update-hnswlib-index--jina-docs.netlify.app 🎉

@JoanFM JoanFM merged commit e24be8d into main Jun 1, 2023
@JoanFM JoanFM deleted the fix-update-hnswlib-index branch June 1, 2023 14:19
@samsja samsja mentioned this pull request Jun 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow update on HNSWLib via reindexing

3 participants