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

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: Needs Review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants