Skip to content

Conversation

@lochmueller
Copy link
Contributor

@lochmueller lochmueller commented Nov 7, 2025

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

This commit adds openrouter.ai features:

  • Add the current set of models (without capabilities, because they are not checked)
  • Add an Embeddings model + client and result converter to support embeddings
  • Add an example for embeddings models
  • Add error handling in the default ResultConverter of the completions endpoint to handle the "rate limit" of the free tier in a better way.

@carsonbot carsonbot added Platform Issues & PRs about the AI Platform component Status: Needs Review labels Nov 7, 2025
@OskarStark OskarStark changed the title [Platform] OpenRouter - Add Embeddings incl. example, fill Model cata… [Platform][OpenRouter] Add Embeddings incl. example, fill Model catalog Nov 7, 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.

Happy to see openrouter get some attention - thanks for that 🙏

Left some minor comments, but close for merging 👍

@lochmueller
Copy link
Contributor Author

@OskarStark thanks for the reviews. What is exactly the problem of the Fabbot pipeline? I have to change the PR header or description.... but in which way? https://symfony.com/doc/current/contributing/code/pull_requests.html#make-a-pull-request Do you have a hint for me? I don't get it...

@chr-hertel
Copy link
Member

I think it's about the actual comment of the PR
image

fixed it, but Fabbot still not happy 🤷‍♂️

@lochmueller
Copy link
Contributor Author

Hey @chr-hertel
just changing the PR do not retrigger the action. I add some error code handling, retrigger the action and it looks right now. Thank you...
Regards,
Tim

@chr-hertel chr-hertel force-pushed the feature/openrouter-embeddings branch from bdd4616 to e9d5bb5 Compare November 12, 2025 22:34
@chr-hertel
Copy link
Member

Thank you @lochmueller.

@chr-hertel chr-hertel merged commit b91d837 into symfony:main Nov 12, 2025
14 checks passed
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