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

Fix llama 3 #1671

Merged
merged 2 commits into from
Apr 24, 2024
Merged

Fix llama 3 #1671

merged 2 commits into from
Apr 24, 2024

Conversation

minhthuc2502
Copy link
Collaborator

No description provided.

@BBC-Esq
Copy link

BBC-Esq commented Apr 23, 2024

Am I correct in understanding that by setting the system prompt at the command line that you're caching the static prompt, or whatever it's called within ctranslate2?

@minhthuc2502
Copy link
Collaborator Author

System prompt is the prompt added after <|start_header_id|>system<|end_header_id|>. It is defined in the template of Llama 3 not a specified thing of ctranslate2.

@minhthuc2502 minhthuc2502 merged commit 042aac7 into OpenNMT:master Apr 24, 2024
17 checks passed
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.

None yet

2 participants