-
Notifications
You must be signed in to change notification settings - Fork 39
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
Provide common interface to common parameters like temperature
#280
Comments
check the seed parameter here (I didn't test yet) |
I'm already setting that. |
https://openrouter.ai/docs/parameters probably a good place to start. And maybe call them parameters? |
Parameters is probably a good naming one thing to be aware of that not all are called the same or available from different providers so some kind of mapping of names and availability could be an option |
Or maybe start with the most important ones temperature and top p |
Probably using an options object.
And then experiment with setting
temperature
to 0 to make tests more reliable.The text was updated successfully, but these errors were encountered: