Skip to content

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