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

Python: added reference to AzureTextCompletion to notebook #1456

Merged
merged 3 commits into from
Jun 14, 2023

Conversation

mathyousee
Copy link
Contributor

Motivation and Context

This is addressing issue#1455 #1455

Description

In the second cell, I added the missing reference to Azure Text Completion.

In the Context Variables section, I updated the first cell to include an option to run using Azure OpenAI (was missed previously). This follows the same format as used in other cells.

@alexchaomander
Copy link
Contributor

Thanks for the fix!

@alexchaomander alexchaomander added the python Pull requests for the Python Semantic Kernel label Jun 14, 2023
@alexchaomander alexchaomander changed the title added reference to AzureTextCompletion Python: added reference to AzureTextCompletion to notebook Jun 14, 2023
@alexchaomander alexchaomander added the PR: ready to merge PR has been approved by all reviewers, and is ready to merge. label Jun 14, 2023
@github-actions github-actions bot removed the python Pull requests for the Python Semantic Kernel label Jun 14, 2023
@lemillermicrosoft lemillermicrosoft enabled auto-merge (squash) June 14, 2023 19:29
@lemillermicrosoft lemillermicrosoft merged commit d5de382 into microsoft:main Jun 14, 2023
shawncal pushed a commit to shawncal/semantic-kernel that referenced this pull request Jul 6, 2023
…#1456)

### Motivation and Context
This is addressing issue#1455
microsoft#1455

### Description
In the second cell, I added the missing reference to Azure Text
Completion.

In the _Context Variables_ section, I updated the first cell to include
an option to run using Azure OpenAI (was missed previously). This
follows the same format as used in other cells.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: ready to merge PR has been approved by all reviewers, and is ready to merge.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants