[PR #10775] [MERGED] ml: add more rope options #23900

Closed
opened 2026-04-19 17:16:45 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/ollama/ollama/pull/10775
Author: @mxyng
Created: 5/19/2025
Status: Merged
Merged: 5/20/2025
Merged by: @mxyng

Base: mainHead: mxyng/rope-options


📝 Commits (1)

  • f16ec26 ml: add more rope options

📊 Changes

11 files changed (+116 additions, -83 deletions)

View changed files

📝 ml/backend.go (+0 -16)
📝 ml/backend/ggml/ggml.go (+7 -19)
ml/nn/fast/rope.go (+21 -0)
ml/nn/rope/rope.go (+33 -0)
📝 model/models/gemma2/model.go (+5 -4)
📝 model/models/gemma3/model_text.go (+5 -4)
📝 model/models/llama/model.go (+9 -8)
📝 model/models/llama4/model_text.go (+5 -3)
📝 model/models/mistral3/model_text.go (+8 -8)
📝 model/models/mllama/model_text.go (+7 -6)
📝 model/models/qwen25vl/model_text.go (+16 -15)

📄 Description

expanding on the idea introduced in #10385 with rope options to also take type and factors


🔄 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/10775 **Author:** [@mxyng](https://github.com/mxyng) **Created:** 5/19/2025 **Status:** ✅ Merged **Merged:** 5/20/2025 **Merged by:** [@mxyng](https://github.com/mxyng) **Base:** `main` ← **Head:** `mxyng/rope-options` --- ### 📝 Commits (1) - [`f16ec26`](https://github.com/ollama/ollama/commit/f16ec2667e648a5b366247e667f266f5ddb619cb) ml: add more rope options ### 📊 Changes **11 files changed** (+116 additions, -83 deletions) <details> <summary>View changed files</summary> 📝 `ml/backend.go` (+0 -16) 📝 `ml/backend/ggml/ggml.go` (+7 -19) ➕ `ml/nn/fast/rope.go` (+21 -0) ➕ `ml/nn/rope/rope.go` (+33 -0) 📝 `model/models/gemma2/model.go` (+5 -4) 📝 `model/models/gemma3/model_text.go` (+5 -4) 📝 `model/models/llama/model.go` (+9 -8) 📝 `model/models/llama4/model_text.go` (+5 -3) 📝 `model/models/mistral3/model_text.go` (+8 -8) 📝 `model/models/mllama/model_text.go` (+7 -6) 📝 `model/models/qwen25vl/model_text.go` (+16 -15) </details> ### 📄 Description expanding on the idea introduced in #10385 with rope options to also take type and factors --- <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 2026-04-19 17:16:45 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/ollama#23900