Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add dynamic openrouter model list #233

Conversation

wonderwhy-er
Copy link
Collaborator

@wonderwhy-er wonderwhy-er commented Nov 9, 2024

Slowly adding features from my fork

List of things that were done:

  • Added getOpenRouterModels to constants.ts to get dynamic list of all OpenRouter models
  • Labels also contain prices and context sizes
  • Made all dynamic lists load in parallel to speed up getting them
  • There was a bug that some models are stuck in model selector no matter what provider is chosen, fixed by chnaging what keys are used for rendering, something was caching wrong in React

Optional: I am not sure about is price and context size added to label.
I think its useful information but its not super readable at the moment.

image

@wonderwhy-er wonderwhy-er changed the title feat: Add dynamic openrouter model list feat: add dynamic openrouter model list Nov 9, 2024
@wonderwhy-er
Copy link
Collaborator Author

@champ2050
Copy link

Could someone kindly review this and merge it? There are several open PRs that need to be reviewed and merged.

@wonderwhy-er
Copy link
Collaborator Author

I just realized there is better place for this code, I will refactor it, no need to review/merge yet

@wonderwhy-er wonderwhy-er marked this pull request as draft November 10, 2024 13:23
@wonderwhy-er wonderwhy-er marked this pull request as ready for review November 10, 2024 14:25
…ter-model-list

# Conflicts:
#	app/utils/constants.ts
…ter-model-list

# Conflicts:
#	app/components/chat/BaseChat.tsx
@wonderwhy-er
Copy link
Collaborator Author

I will merge it with my other API Keys related PR, will close this one

@wonderwhy-er
Copy link
Collaborator Author

Closed as it was merged with #251

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants