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

Improve Copilot + local AI setup instructions #7413

Merged
merged 6 commits into from
Oct 10, 2024
Merged

Improve Copilot + local AI setup instructions #7413

merged 6 commits into from
Oct 10, 2024

Conversation

azigler
Copy link
Contributor

@azigler azigler commented Sep 18, 2024

I noticed this comment from a user that had trouble configuring Copilot with Ollama and realized that the documentation instructions do not include the specific URL instructions to access the OpenAI-compatible API. This PR clarifies that instruction, in line with what's taught in the Academy course.

@azigler azigler added 1: Dev Review Requires review by a core commiter 2: Editor Review Requires review by an editor labels Sep 18, 2024
@azigler azigler self-assigned this Sep 18, 2024
cwarnermm
cwarnermm approved these changes Sep 18, 2024
View reviewed changes
Copy link
Member

@cwarnermm cwarnermm left a comment

Choose a reason for hiding this comment

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

Thank you, @azigler!

source/configure/enable-copilot.rst Outdated Show resolved Hide resolved
@cwarnermm cwarnermm removed the 2: Editor Review Requires review by an editor label Sep 18, 2024
@github-actions GitHub Actions
Copy link

Newest code from mattermost has been published to preview environment for Git SHA bfa8ac9

1 similar comment
@github-actions GitHub Actions
Copy link

Newest code from mattermost has been published to preview environment for Git SHA bfa8ac9

@azigler
Copy link
Contributor Author

azigler commented Sep 18, 2024

@crspeller Should we include a recommendation for disabling tools? Is that a broad recommendation? If so, should we revisit the option and it's default setting? As a minor note, it's a bit confusing that the field is "disable" and has a true/false (which means the radio toggle is semantically opposite from the end result, which might confuse users). What about "Enable Tools" and it has a default setting to whatever you think is most appropriate?

@azigler azigler mentioned this pull request Sep 20, 2024
Open
@cwarnermm
Copy link
Member

Friendly reminder that this docs PR will benefit from your review, @crspeller.

@github-actions GitHub Actions
Copy link

github-actions bot commented Oct 2, 2024

Newest code from mattermost has been published to preview environment for Git SHA 1793fae

@github-actions GitHub Actions
Copy link

github-actions bot commented Oct 2, 2024

Newest code from mattermost has been published to preview environment for Git SHA bcdd240

crspeller
crspeller requested changes Oct 3, 2024
View reviewed changes
source/configure/enable-copilot.rst Outdated Show resolved Hide resolved
@cwarnermm
Copy link
Member

cwarnermm commented Oct 9, 2024 •
edited
Loading

@azigler - Are you open to adding a direct link from our LLM admin docs to LLM setup training via Academy? You can use #7465 as a reference to do so. The Academy link is: https://academy.mattermost.com/p/new-mattermost-copilot-multi-llm-setup-usage (or convert to a PL link)

@github-actions GitHub Actions
Copy link

github-actions bot commented Oct 9, 2024

Newest code from mattermost has been published to preview environment for Git SHA 7086fa8

@azigler
📝 (academy-copilot-setup.rst): Add new file to include Mattermost Aca…
07ab3dc 
…demy badge

📝 (enable-copilot.rst): Include Mattermost Academy badge and update API URL instructions
The new file academy-copilot-setup.rst was added to include a Mattermost Academy badge that links to a tutorial on setting up and configuring Mattermost Copilot with multiple LLMs. This badge was then included in enable-copilot.rst to provide users with a direct link to the tutorial. The instructions for entering the API URL were also updated to clarify the URL format for services other than Ollama.
@github-actions GitHub Actions
Copy link

github-actions bot commented Oct 9, 2024

Newest code from mattermost has been published to preview environment for Git SHA 07ab3dc

@azigler
Copy link
Contributor Author

azigler commented Oct 9, 2024

@crspeller Clarified that.

@cwarnermm Added that!

Good to go! 👍

@azigler azigler requested a review from crspeller October 9, 2024 19:08
@cwarnermm cwarnermm added 3: Reviews Complete All reviewers have approved the pull request and removed 1: Dev Review Requires review by a core commiter labels Oct 10, 2024
@cwarnermm cwarnermm merged commit 13a58a4 into master Oct 10, 2024
4 checks passed
@cwarnermm cwarnermm deleted the azigler-patch-1 branch October 10, 2024 12:58
@github-actions GitHub Actions
Copy link

Newest code from mattermost has been published to preview environment for Git SHA 0be5686

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cwarnermm cwarnermm cwarnermm approved these changes

@crspeller crspeller crspeller approved these changes

Assignees

@azigler azigler

Labels
3: Reviews Complete All reviewers have approved the pull request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@azigler @cwarnermm @crspeller