Skip to content

Conversation

@szabosteve
Copy link
Contributor

@szabosteve szabosteve commented Nov 24, 2025

Summary

Related to https://github.com/elastic/docs-content-internal/issues/528

This PR:

  • adds a new section under Built-in NLP Models called Jina
  • adds docs about the jina-embeddings-v3 model to the new page
  • adds an applies_to tag to the model documentation section (tech preview in serverless, tech preview in 9.3 – it will be rendered as "Planned" on the doc page without explicit reference to a certain version number)

TO DO

  • Performance considerations

Generative AI disclosure

  1. Did you use a generative AI (GenAI) tool to assist in creating this contribution?
  • Yes
  • No

Notes

Benchmarking info will be added in a separate PR.

@szabosteve szabosteve added documentation Improvements or additions to documentation nlp inference eis Team:Developer Issues owned by the Developer Docs Team labels Nov 24, 2025
@github-actions
Copy link

github-actions bot commented Nov 24, 2025

Vale Linting Results

Summary: 2 suggestions found

💡 Suggestions (2)
File Line Rule Message
explore-analyze/machine-learning/nlp/ml-nlp-jina.md 17 Elastic.Capitalization '[jina-embeddings-v3]' should use sentence-style capitalization.
explore-analyze/machine-learning/nlp/ml-nlp-jina.md 27 Elastic.Acronyms 'ELSER' has no definition.

@github-actions
Copy link

github-actions bot commented Nov 24, 2025

Copy link
Contributor

@demjened demjened left a comment

Choose a reason for hiding this comment

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

LGTM, added some text for the performance considerations section. For benchmarking results please hold off until https://github.com/elastic/search-team/issues/11721 is complete.

@szabosteve szabosteve marked this pull request as ready for review December 2, 2025 11:13
@szabosteve szabosteve requested review from a team as code owners December 2, 2025 11:13
Copy link
Contributor

@kosabogi kosabogi left a comment

Choose a reason for hiding this comment

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

LGTM with 3 small suggestions

Copy link
Contributor

@florent-leborgne florent-leborgne left a comment

Choose a reason for hiding this comment

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

LGTM, just a few optional suggestions

Comment on lines +12 to +13
This page collects all Jina models you can use as part of the {{stack}}.
Currently, the following models are available as built-in models:
Copy link
Contributor

Choose a reason for hiding this comment

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

should we surface the "machine learning" context more clearly as part of this intro? or add a description to the frontmatter that does that?

szabosteve and others added 3 commits December 2, 2025 15:29
Co-authored-by: florent-leborgne <florent.leborgne@elastic.co>
Co-authored-by: florent-leborgne <florent.leborgne@elastic.co>
@szabosteve szabosteve merged commit 9a59b34 into main Dec 2, 2025
8 checks passed
@szabosteve szabosteve deleted the szabosteve/jinaAI branch December 2, 2025 16:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation eis inference nlp Team:Developer Issues owned by the Developer Docs Team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants