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

The types of Temperature & TopP are ints but need to take floats #12

Open
LaraSQP opened this issue Oct 21, 2023 · 2 comments
Open

The types of Temperature & TopP are ints but need to take floats #12

LaraSQP opened this issue Oct 21, 2023 · 2 comments

Comments

@LaraSQP
Copy link

LaraSQP commented Oct 21, 2023

First of all, thank you very much for developing this library and making it available to all.

Now, the types of Temperature & TopP fields in CompletionCreateOptions are declared as int? yet the documentation itself of those fields (in the library) says that they should/could take values like 0.1 or 0.9. This agrees with the official OpenAI docs too, btw.

There seems to be an error here in the choice of type but the error could well be mine. Please explain.

Thank you.

@Netizine
Copy link
Owner

Youre right. Ill update the SDK tomorrow.

@LaraSQP
Copy link
Author

LaraSQP commented Oct 22, 2023

Please update the NuGet in the process.

Much appreciated.

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