Skip to content

Conversation

@lochmueller
Copy link
Contributor

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
// In case free is running into 429 rate limit errors, you can use the paid model:
// $model = 'google/gemini-2.0-flash-lite-001';
$model = 'google/gemini-2.0-flash-exp:free';
// In case free is running into 429 rate limit errors (note: it is a 404 error code in openrouter api), you can use the paid model:
Copy link
Member

Choose a reason for hiding this comment

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

if 429 is not accurate, just adopt it

Suggested change
// In case free is running into 429 rate limit errors (note: it is a 404 error code in openrouter api), you can use the paid model:
// In case free is running into 404 errors, you can use the paid model:

// Models are dynamically available and identified by provider/model format
// Capabilities are not set in this model catalog

// Modelle
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
// Modelle
// Models

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 👍

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 Work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants