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

Added LangChain examples to MD doc #521

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

dipatidar
Copy link
Contributor

No description provided.

@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Dec 2, 2024
darenr
darenr previously approved these changes Dec 3, 2024
ai-quick-actions/model-deployment-tips.md Outdated Show resolved Hide resolved
)

#Invocation
messages = [
Copy link
Contributor

Choose a reason for hiding this comment

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

This example does not work for Mistral model as it does not take system message. Maybe we should mention this as well. There are some discussions: vllm-project/vllm#2112

Copy link
Contributor Author

Choose a reason for hiding this comment

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

As suggested in the latest conversation in the above thread, I tried the MD endpoint for Mistral-7B-Instruct-v0.3, and it's working with the same sample code.

Copy link
Contributor

Choose a reason for hiding this comment

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

Which conversation are your referring to? It does not seem to work for me in with vllm and the Mistral-7B-Instruct-v0.3 model.

Copy link
Contributor

Choose a reason for hiding this comment

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

Looks like you are using a newer version of the model with the updated chat template. The model provided by the service 83e9aa1 does not support system message. For this to work, users will have to download the newer version instead of using the one provided by the service.

mrDzurb
mrDzurb previously approved these changes Dec 3, 2024
Copy link
Contributor

@kumar-shivam-ranjan kumar-shivam-ranjan left a comment

Choose a reason for hiding this comment

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

Do we have langchain code snippet for embedding models as well?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OCA Verified All contributors have signed the Oracle Contributor Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants