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.

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

AI Assistant image updates, edits from twin PR, model recommendation #5237

Merged
merged 7 commits into from
May 21, 2024
Merged

AI Assistant image updates, edits from twin PR, model recommendation #5237

merged 7 commits into from
May 21, 2024

Conversation

benironside
Copy link
Contributor

@benironside benironside commented May 20, 2024 •
edited
Loading

Fixes #5040. This is a twin PR of #344, except two differences:

  • The connector setup guides were already present in ESS docs, so weren't added here.
  • This PR updates the "Recommended models" section - so this is the only text in need of a review. The rest was already reviewed in the twin PR.

Preview: AI Assistant

@benironside benironside added Feature: Elastic AI Assistant Interface for interacting with generative AIs v8.14.0 labels May 20, 2024
@benironside benironside self-assigned this May 20, 2024
@benironside benironside requested a review from a team as a code owner May 20, 2024 19:25
@github-actions GitHub Actions
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.

@natasha-moore-elastic natasha-moore-elastic mentioned this pull request May 21, 2024
Closed
22 tasks
natasha-moore-elastic
natasha-moore-elastic previously approved these changes May 21, 2024
View reviewed changes
Copy link
Contributor

@natasha-moore-elastic natasha-moore-elastic 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 one question about the Recommended models section.

@benironside benironside requested a review from a team May 21, 2024 17:59
jmikell821
jmikell821 approved these changes May 21, 2024
View reviewed changes
Copy link
Contributor

@jmikell821 jmikell821 left a comment

Choose a reason for hiding this comment

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

LGTM!

@benironside benironside merged commit cdda1d2 into main May 21, 2024
3 checks passed
mergify bot pushed a commit that referenced this pull request May 21, 2024
@benironside @mergify
AI Assistant image updates, edits from twin PR, model recommendation (#…
f19db32 
…5237)

* Image updates, edits from twin PR

* updates recommended models section with Dhru's feedback

* updates image

* minor edit

* minor edit

* updates model recc section

(cherry picked from commit cdda1d2)
@mergify mergify bot mentioned this pull request May 21, 2024
Merged
benironside added a commit that referenced this pull request May 23, 2024
@mergify @benironside
AI Assistant image updates, edits from twin PR, model recommendation (#…
dd23ce6 
…5237) (#5254)

* Image updates, edits from twin PR

* updates recommended models section with Dhru's feedback

* updates image

* minor edit

* minor edit

* updates model recc section

(cherry picked from commit cdda1d2)

Co-authored-by: Benjamin Ironside Goldstein <91905639+benironside@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jmikell821 jmikell821 jmikell821 approved these changes

@natasha-moore-elastic natasha-moore-elastic natasha-moore-elastic left review comments

Assignees

@benironside benironside

Labels
Feature: Elastic AI Assistant Interface for interacting with generative AIs v8.14.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Request] AI Assistant Redesigned Look
3 participants
@benironside @jmikell821 @natasha-moore-elastic