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

Chunking notebooks: mention semantic_text #280

Merged
merged 3 commits into from
Sep 18, 2024

Conversation

maxjakob
Copy link
Contributor

Raise awareness for the semantic_text field type in the notebooks related to chunking.

Copy link

gitnotebooks bot commented Jun 25, 2024

Found 3 changed notebooks. Review the changes at https://gitnotebooks.com/elastic/elasticsearch-labs/pull/280

@maxjakob
Copy link
Contributor Author

wait for 8.15 to merge this

"\n",
"**Elasticsearch version 8.15 introduced the [`semantic_text`](https://www.elastic.co/guide/en/elasticsearch/reference/master/semantic-text.html) field type which handles the chunking process behind the scenes. Before continuing with this notebook, we highly recommend looking into this:**\n",
"\n",
"**<https://www.elastic.co/search-labs/blog/semantic-search-simplified-semantic-text>**"
Copy link
Contributor Author

@maxjakob maxjakob Sep 17, 2024

Choose a reason for hiding this comment

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

@SharonRosencwaig1 @ElishevaStern Do we usually add tracking parameters for links from notebooks to Search Labs posts?

@joemcelroy
Copy link
Member

we also have a semantic_text notebook that would be good to link aswell https://github.com/elastic/elasticsearch-labs/blob/main/notebooks/search/09-semantic-text.ipynb

@maxjakob maxjakob merged commit b59f3c7 into main Sep 18, 2024
5 checks passed
@maxjakob maxjakob deleted the tokenization-notebook-semantic-text branch September 18, 2024 08:51
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