Skip to content

[Solutions] Adds info about preconfigured LLM #955

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 10 commits into from
Apr 1, 2025
Merged

[Solutions] Adds info about preconfigured LLM #955

merged 10 commits into from
Apr 1, 2025

Conversation

szabosteve
Copy link
Contributor

@szabosteve szabosteve commented Mar 27, 2025 •
edited
Loading

Overview

This PR adds info about the preconfigured LLM to the Playground and AI Assistant docs.

@szabosteve szabosteve added documentation Improvements or additions to documentation nlp eis labels Mar 27, 2025
leemthompo
leemthompo approved these changes Mar 27, 2025
View reviewed changes
Copy link
Contributor

@leemthompo leemthompo 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 very minor optional suggestion for parallelism :)

@shubhaat shubhaat assigned shubhaat and unassigned shubhaat Apr 1, 2025
shubhaat
shubhaat suggested changes Apr 1, 2025
View reviewed changes
Copy link
Contributor

@shubhaat shubhaat left a comment

Choose a reason for hiding this comment

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

We want to not use the term "Elastic LLM" for now and use a more general the preconfigured connector or preconfigured LLM where possible to be accurate.

@szabosteve szabosteve marked this pull request as ready for review April 1, 2025 15:10
@szabosteve szabosteve changed the title [Solutions] Adds info about Elastic LLM [Solutions] Adds info about preconfigured LLM Apr 1, 2025
shubhaat
shubhaat reviewed Apr 1, 2025
View reviewed changes
szabosteve
szabosteve commented Apr 1, 2025
View reviewed changes
@szabosteve szabosteve requested a review from shubhaat April 1, 2025 15:19
@szabosteve szabosteve merged commit 5b26c68 into main Apr 1, 2025
4 checks passed
@szabosteve szabosteve deleted the szabosteve/elastic-llm-pg-aia branch April 1, 2025 15:33
@szabosteve szabosteve self-assigned this Apr 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@leemthompo leemthompo leemthompo approved these changes

+1 more reviewer

@shubhaat shubhaat shubhaat approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@szabosteve szabosteve

Labels

documentation Improvements or additions to documentation eis nlp

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@szabosteve @shubhaat @leemthompo