set the maximum value of Top p parameter to 1

#2
by SaulLu - opened

My understanding was that the API was using the arguments for the generations defined here, if this is the case, shouldn't the maximum value of top_p be 1?

Hugging Face org

I think top_p is now hardcoded, would you be OK with closing this PR?

clefourrier changed pull request status to closed

Sign up or log in to comment