Skip to content

added temperature to the tutorial documentation - #415

Merged
mmabrouk merged 2 commits into
Agenta-AI:mainfrom
philipokiokio:tutorial_pr
Aug 20, 2023
Merged

mmabrouk merged 2 commits into
Agenta-AI:mainfrom
philipokiokio:tutorial_pr

Conversation

@philipokiokio

Copy link
Copy Markdown
Contributor

In the tutorial documentation, the temperature was not explicitly stated.

although there was a default value set in the code.

I provided temperature and a little comment on what temperature does.

…he first input as default rather than a list of choices, so using the choices argument takes care of the error thrown.
@philipokiokio

Copy link
Copy Markdown
Contributor Author

I also got an error following the documentation because the MultipleChoiceParam that was used did not use arguments. So the list provided was automatically assigned to the default. Using choices arg takes care of that issue.

@mmabrouk mmabrouk left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @philipokiokio for the PR!

@mmabrouk
mmabrouk merged commit effe94c into Agenta-AI:main Aug 20, 2023
@mmabrouk

Copy link
Copy Markdown
Member

@all-contributors please add @philipokiokio for doc

@allcontributors

Copy link
Copy Markdown
Contributor

@mmabrouk

I've put up a pull request to add @philipokiokio! 🎉

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.

2 participants