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

add url parameter for assistant to docs #572

Merged
merged 4 commits into from
Nov 2, 2024

Conversation

thibaultmol
Copy link
Contributor

I also would have not known it was a thing if I hadn't seen it on discord. but it's not actually mentioned in the docs yet

https://kagifeedback.org/d/5254-hyperlink-directly-to-specific-assistants

@thibaultmol
Copy link
Contributor Author

PLEASE CHECK THE ASSISTANT LIST before merging. There are mistakes in there, I just don't know what the right ones are atm

@david-ros
Copy link
Collaborator

@thibaultmol I think you didn't mean to include this in your commit:

For this addition, a natural location would be right after the LLMs Available in The Assistant section, adding a clear explanation on the use of the profile parameter to select a specific LLM, like gpt-4o or claude3.5. Here's the updated content for that section:

@thibaultmol
Copy link
Contributor Author

yep, had also noticed it right before you mentioned it xd
not sure why it didn't build before, hoping it builds now.

But yeah, someone should still check and change the exact profile names because some of them are def not correct

@thibaultmol thibaultmol marked this pull request as draft November 1, 2024 19:24
@thibaultmol
Copy link
Contributor Author

Feel free to edit whatever and then merge

@nobodywasishere
Copy link
Member

I've updated the profile names and added more docs on other parameters. I'll leave it to @david-ros to approve / merge.

@david-ros david-ros marked this pull request as ready for review November 2, 2024 04:36
@david-ros david-ros merged commit 0b4f005 into kagisearch:main Nov 2, 2024
1 check passed
@thibaultmol
Copy link
Contributor Author

thibaultmol commented Nov 2, 2024

Probably a dumb thing: i had put gpt4o first because that's the recommended model of all the openai's ones (people might instinctively pick the first one in the list even though that's not the best one. And it's the most expensive one(

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.

3 participants