[PR #8055] [MERGED] llama: enable JSON schema key ordering for generating grammars #17877

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

📋 Pull Request Information

Original PR: https://github.com/ollama/ollama/pull/8055
Author: @ParthSareen
Created: 12/11/2024
Status: Merged
Merged: 12/12/2024
Merged by: @ParthSareen

Base: mainHead: parth/fix-json-ordering-SO


📝 Commits (1)

  • 023fc86 Enable JSON schema ordering

📊 Changes

3 files changed (+24 additions, -6 deletions)

View changed files

📝 llama/grammar_test.go (+1 -5)
📝 llama/json-schema-to-grammar.cpp (+1 -1)
llama/patches/0012-Maintain-ordering-for-rules-for-grammar.patch (+22 -0)

📄 Description

Will do a follow up PR for updates to the command line with format


🔄 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/8055 **Author:** [@ParthSareen](https://github.com/ParthSareen) **Created:** 12/11/2024 **Status:** ✅ Merged **Merged:** 12/12/2024 **Merged by:** [@ParthSareen](https://github.com/ParthSareen) **Base:** `main` ← **Head:** `parth/fix-json-ordering-SO` --- ### 📝 Commits (1) - [`023fc86`](https://github.com/ollama/ollama/commit/023fc86c1ba81e4c27b91aeeb43385ef507be123) Enable JSON schema ordering ### 📊 Changes **3 files changed** (+24 additions, -6 deletions) <details> <summary>View changed files</summary> 📝 `llama/grammar_test.go` (+1 -5) 📝 `llama/json-schema-to-grammar.cpp` (+1 -1) ➕ `llama/patches/0012-Maintain-ordering-for-rules-for-grammar.patch` (+22 -0) </details> ### 📄 Description Will do a follow up PR for updates to the command line with format --- <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-16 06:17:42 -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#17877