Skip to content

Conversation

taras-yemets
Copy link
Contributor

@taras-yemets taras-yemets commented Oct 28, 2024

Closes https://linear.app/sourcegraph/issue/PRIME-522/modelconfiguration-arrive-at-a-nice-tutorial-like-docs-structure

Adds docs on the "modelConfiguration" field in the site configuration, including examples.

Copy link

vercel bot commented Oct 28, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
sourcegraph-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 25, 2024 11:48pm

Copy link
Contributor

@aramaraju aramaraju left a comment

Choose a reason for hiding this comment

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

In general, we should strive to have the configuration examples for the known deployments, with our recommendations. For example: List out the config example for AWS Bedrock where the Claude 3.5 Sonnet v2 is selected as Chat, and Haiku as autocomplete. This will be very useful for admins to simply copy paste and not have to think about it

@MaedahBatool
Copy link
Contributor

@aramaraju @taras-yemets thanks for this PR. It has all the good content. Quick question is there a hard deadline for when we need to publish it?

We are doing a docs hackathon for Cody next week so I was wondering if we can merge this PR after that so we can do a final cleanup first. WDYT?

@taras-yemets
Copy link
Contributor Author

We’re missing some details regarding self-hosted models and the OpenAI-compatible server-side configuration. To avoid blocking the merge, I’ve removed these sections from this PR. We can address the missing information in a follow-up PR.
cc: @aramaraju, @slimsag

@aramaraju
Copy link
Contributor

We’re missing some details regarding self-hosted models and the OpenAI-compatible server-side configuration. To avoid blocking the merge, I’ve removed these sections from this PR. We can address the missing information in a follow-up PR. cc: @aramaraju, @slimsag

I'm completely okay and very supportive of that!

@emidoots
Copy link
Member

Thank you @taras-yemets ! Super supportive of moving forward as-is, and so sorry it's taken me forever to get around to this!

Copy link
Member

@emidoots emidoots left a comment

Choose a reason for hiding this comment

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

General approval

MaedahBatool and others added 2 commits November 25, 2024 13:49
Co-authored-by: David Veszelovszki <[email protected]>
@MaedahBatool
Copy link
Contributor

Thanks a ton to each and everyone who contributed to this PR. Keeping the release timeline. I'll be merging this PR. Any new follow-up or improvements will be tackled in a new PR. Appreciate all your help here. ❤️

Copy link
Contributor

@MaedahBatool MaedahBatool left a comment

Choose a reason for hiding this comment

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

Let's go 🚀

@MaedahBatool MaedahBatool changed the base branch from main to sg-next-nov25 November 25, 2024 23:45
@MaedahBatool MaedahBatool merged commit 54b5da2 into sg-next-nov25 Nov 26, 2024
5 checks passed
@MaedahBatool MaedahBatool deleted the ty/modelconfig branch November 26, 2024 00:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants