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

when conducting the inference, do you set do_sample=True? #1

Open
renatz opened this issue Mar 19, 2024 · 1 comment
Open

when conducting the inference, do you set do_sample=True? #1

renatz opened this issue Mar 19, 2024 · 1 comment

Comments

@renatz
Copy link

renatz commented Mar 19, 2024

for the temperature non-zero, i think you set do_sample=True?
how about the top_p setting?

@grayground
Copy link
Collaborator

Thank you for your question.
For closed-source models, increasing the temperature parameter essentially equates to performing sampling (as evidenced by the results in the paper's section "The Impact of Sampling Temperature").
For open-source models, we did not specifically adjust the top_p parameter; in vicuna-7b-v1.5, the default value for the top_p parameter is 0.6.

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

No branches or pull requests

2 participants