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

Improve styling for sampling parameters. #446

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mkbehr
Copy link
Collaborator

@mkbehr mkbehr commented Feb 7, 2025

Add a new name-value-input for custom body request fields. Move each numerical sampling parameter into its own number-input div so they're not as wide.

Also set the minimum of the frequency penalty and presence penalty boxes to 0; this makes them equal in width to the others, and it's unlikely that a user would want to set them below 0. (If they really want to type a negative number, I think it will work anyway.)

Add a new name-value-input for custom body request fields. Move each numerical
sampling parameter into its own number-input div so they're not as wide.

Also set the minimum of the frequency penalty and presence penalty boxes to 0;
this makes them equal in width to the others, and it's unlikely that a user
would want to set them below 0. (If they really want to type a negative number,
I think it will work anyway.)
@mkbehr mkbehr requested a review from vivtsai February 7, 2025 20:31
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.

1 participant