-
Notifications
You must be signed in to change notification settings - Fork 138
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
[enhancement] Provide information about the different models #315
Comments
Wow this looks great! Please give me a few days to take a look at your code, it's been a rough week. This ties in with some refactoring of the backend definitions I've been meaning to do that should make attaching metadata to the models easier. |
@benthamite As of gptel 0.9.5 (published today),
Now would be a good time to incorporate your suggested changes. The only change from your version is that instead of defining a separate |
Thanks! Let me take a closer look and I’ll try to prepare a pull request. |
@karthink, a couple of questions:
|
The last step will be updating the annotation function for the model selector in gptel-transient to show these details when selecting a model. |
Considering the amount of information that will be shoved into the definition of (defconst gptel--anthropic-models
;; model details here
)
(cl-defun gptel-make-anthropic (name &key (models gptel--anthropic-models) ...) ...) |
Makes sense, thanks. Concerning (3), yeah, sorry: I meant to suggest that I could add the |
Noted, thanks. |
Closed by #420. |
In my personal config I have some extensions for
gptel
, including a function and associated variable that displays information about the available models:You can find it here. Would you be interested in a pull request? (I would adapt some of the descriptions to make them less subjective and personal.)
The text was updated successfully, but these errors were encountered: