Skip to content

Conversation

Guikingone
Copy link
Contributor

@Guikingone Guikingone commented Aug 11, 2025

Q A
Bug fix? Kind of
New feature? no
Docs? yes
Issues Nope
License MIT

Hi 👋🏻

This PR aims to introduce the support for structured output via Ollama, the feature is fully functional since a few versions ans by default, the client does not map the "format" to the HTTP request, here's the fix/feature.

@carsonbot carsonbot added Status: Needs Review Platform Issues & PRs about the AI Platform component labels Aug 11, 2025
@carsonbot carsonbot changed the title [Platform]: Ollama structured output support [Platform] : Ollama structured output support Aug 11, 2025
@Guikingone Guikingone force-pushed the ollama/structure_output branch from 6f67c91 to 4fbbb14 Compare August 11, 2025 14:23
@chr-hertel chr-hertel changed the title [Platform] : Ollama structured output support [Platform] Ollama structured output support Aug 11, 2025
Copy link
Member

@chr-hertel chr-hertel left a comment

Choose a reason for hiding this comment

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

Looks quite good, only minor stuff :)

@chr-hertel chr-hertel force-pushed the ollama/structure_output branch from 7025160 to 11c2d17 Compare August 12, 2025 07:05
@chr-hertel
Copy link
Member

Thank you @Guikingone.

@chr-hertel chr-hertel merged commit da9a6e3 into symfony:main Aug 12, 2025
7 checks passed
@Guikingone Guikingone deleted the ollama/structure_output branch August 12, 2025 07:06
@OskarStark
Copy link
Contributor

Nice, thanks 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Platform Issues & PRs about the AI Platform component Status: Reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants