[PR #5620] [MERGED] update embedded templates #10594

Closed
opened 2025-11-12 15:32:07 -06:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ollama/ollama/pull/5620
Author: @mxyng
Created: 7/10/2024
Status: Merged
Merged: 7/11/2024
Merged by: @mxyng

Base: mainHead: mxyng/templates


📝 Commits (2)

  • 41be280 add system prompt to first legacy template
  • 19753c1 update embedded templates

📊 Changes

29 files changed (+214 additions, -88 deletions)

View changed files

📝 server/prompt_test.go (+1 -1)
📝 server/routes_create_test.go (+2 -2)
📝 template/alfred.gotmpl (+2 -2)
📝 template/alpaca.gotmpl (+5 -3)
📝 template/chatml.gotmpl (+2 -2)
📝 template/chatqa.gotmpl (+4 -3)
📝 template/codellama-70b-instruct.gotmpl (+5 -5)
📝 template/falcon-instruct.gotmpl (+7 -5)
📝 template/gemma-instruct.gotmpl (+4 -3)
📝 template/granite-instruct.gotmpl (+4 -4)
📝 template/llama2-chat.gotmpl (+4 -4)
📝 template/llama3-instruct.gotmpl (+2 -2)
📝 template/magicoder.gotmpl (+3 -2)
📝 template/mistral-instruct.gotmpl (+3 -2)
📝 template/openchat.gotmpl (+6 -6)
📝 template/phi-3.gotmpl (+2 -2)
📝 template/solar-instruct.gotmpl (+4 -3)
📝 template/starcoder2-instruct.gotmpl (+2 -3)
📝 template/template.go (+90 -11)
📝 template/template_test.go (+47 -14)

...and 9 more files

📄 Description

add test to ensure legacy and messages template produce the same output. there are some templates which cannot produce the same outputs so those will be skipped


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/ollama/ollama/pull/5620 **Author:** [@mxyng](https://github.com/mxyng) **Created:** 7/10/2024 **Status:** ✅ Merged **Merged:** 7/11/2024 **Merged by:** [@mxyng](https://github.com/mxyng) **Base:** `main` ← **Head:** `mxyng/templates` --- ### 📝 Commits (2) - [`41be280`](https://github.com/ollama/ollama/commit/41be28096aa597ded1ef91774ba3e6dfc0a8ccbb) add system prompt to first legacy template - [`19753c1`](https://github.com/ollama/ollama/commit/19753c18c01183b4c974e36e89b0c7cbdcc3c38a) update embedded templates ### 📊 Changes **29 files changed** (+214 additions, -88 deletions) <details> <summary>View changed files</summary> 📝 `server/prompt_test.go` (+1 -1) 📝 `server/routes_create_test.go` (+2 -2) 📝 `template/alfred.gotmpl` (+2 -2) 📝 `template/alpaca.gotmpl` (+5 -3) 📝 `template/chatml.gotmpl` (+2 -2) 📝 `template/chatqa.gotmpl` (+4 -3) 📝 `template/codellama-70b-instruct.gotmpl` (+5 -5) 📝 `template/falcon-instruct.gotmpl` (+7 -5) 📝 `template/gemma-instruct.gotmpl` (+4 -3) 📝 `template/granite-instruct.gotmpl` (+4 -4) 📝 `template/llama2-chat.gotmpl` (+4 -4) 📝 `template/llama3-instruct.gotmpl` (+2 -2) 📝 `template/magicoder.gotmpl` (+3 -2) 📝 `template/mistral-instruct.gotmpl` (+3 -2) 📝 `template/openchat.gotmpl` (+6 -6) 📝 `template/phi-3.gotmpl` (+2 -2) 📝 `template/solar-instruct.gotmpl` (+4 -3) 📝 `template/starcoder2-instruct.gotmpl` (+2 -3) 📝 `template/template.go` (+90 -11) 📝 `template/template_test.go` (+47 -14) _...and 9 more files_ </details> ### 📄 Description add test to ensure legacy and messages template produce the same output. there are some templates which cannot produce the same outputs so those will be skipped --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the
pull-request
label 2025-11-12 15:32:07 -06:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/ollama-ollama#10594
No description provided.