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

[8.16] Adds LLM matrix for OSS models (backport #6424) #6461

Merged
merged 2 commits into from
Jan 21, 2025

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Jan 21, 2025

Fixes #6396 — adds a second matrix to the LLM performance matrix page that describes the performance of open-source models on our tasks. Also flips the existing matrix rows and columns so it can fit better in a browser window (minimize the need to scroll left-right).

Finally, makes a minor addition to the bring your own LLM guide, linking to the performance matrix since it now includes info about open source models.

Previews:

* First draft

* Update llm-performance-matrix.asciidoc

* links from BYO page to performance matrix

* updates table format

* fixes broken ref

* inverts first matrix

* flips tables

* removes old tables

* incorporates last min edit from James

(cherry picked from commit 6d64b54)

# Conflicts:
#	docs/serverless/AI-for-security/connect-to-byo-llm.asciidoc
#	docs/serverless/AI-for-security/llm-performance-matrix.asciidoc
@mergify mergify bot requested a review from a team as a code owner January 21, 2025 20:31
Copy link
Contributor Author

mergify bot commented Jan 21, 2025

Cherry-pick of 6d64b54 has failed:

On branch mergify/bp/8.16/pr-6424
Your branch is up to date with 'origin/8.16'.

You are currently cherry-picking commit 6d64b54e.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:
	modified:   docs/AI-for-security/connect-to-byo.asciidoc
	modified:   docs/AI-for-security/llm-performance-matrix.asciidoc

Unmerged paths:
  (use "git add/rm <file>..." as appropriate to mark resolution)
	deleted by us:   docs/serverless/AI-for-security/connect-to-byo-llm.asciidoc
	deleted by us:   docs/serverless/AI-for-security/llm-performance-matrix.asciidoc

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

Copy link

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.

@benironside benironside merged commit 930c00c into 8.16 Jan 21, 2025
3 checks passed
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.

1 participant