elastic / stack-docs

Elastic Stack Documentation
Other
95 stars 245 forks source link

Fix links to E5 optimized version #2716

Closed pquentin closed 3 months ago

pquentin commented 3 months ago

We maybe should do more, such as explain that linux-x86-64 is indeed the version optimized for "Intel® silicon".

github-actions[bot] commented 3 months ago

A documentation preview will be available soon.

Request a new doc build by commenting * Rebuild this PR: `run docs-build` * Rebuild this PR and all Elastic docs: `run docs-build rebuild` `run docs-build` is much faster than `run docs-build rebuild`. A `rebuild` should only be needed in rare situations. If your PR continues to fail for an unknown reason, the doc build pipeline may be broken. Elastic employees can check the pipeline status [here](https://buildkite.com/elastic/docs-build).
mergify[bot] commented 3 months ago

This pull request does not have a backport label. Could you fix it @pquentin? 🙏 To fixup this pull request, you need to add the backport labels for the needed branches, such as:

pquentin commented 3 months ago

I don't have the permission to add backports labels. Ideally this should be backported to 8.12, 8.13 and 8.14, since #2666 is on those versions, and the files it references are no longer reachable.

davidkyle commented 3 months ago

We maybe should do more, such as explain that linux-x86-64 is indeed the version optimized for "Intel® silicon".

It is mentioned in the docs https://www.elastic.co/guide/en/machine-learning/current/ml-nlp-e5.html

E5 has two versions: one cross-platform version which runs on any hardware and one version which is optimized for Intel® silicon. The Model Management > Trained Models page shows you which version of E5 is recommended to deploy based on your cluster’s hardware.

pquentin commented 3 months ago

Right, I meant it's maybe not obvious that linux-x86-64 is the optimized version since it's not in the name anymore. But the docs now reads "For the optimized version, you need the following files in your system: ..." too so that's fine! Sorry for the noise.

Would you mind adding the backport labels? 8.12, 8.13 and 8.14

pquentin commented 3 months ago

We also need a review from the docs team.

pquentin commented 3 months ago

ping

szabosteve commented 3 months ago

I've also added the backport labels.

pquentin commented 3 months ago

Thanks! I can't merge myself.