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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Consistent Defaults in Python Notebooks #1411

Merged

Conversation

robinmanuelthiel
Copy link
Member

@robinmanuelthiel robinmanuelthiel commented Jun 11, 2023

Motivation and Context

Description

Consistently use useAzureOpenAI = False in Python Notebooks. I stumbled over this. Default was always false.

Contribution Checklist

Copy link
Contributor

@alexchaomander alexchaomander left a comment

Choose a reason for hiding this comment

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

Thanks for adding this to make the notebooks more consistent!

@alexchaomander alexchaomander added python Pull requests for the Python Semantic Kernel PR: ready to merge PR has been approved by all reviewers, and is ready to merge. labels Jun 12, 2023
@lemillermicrosoft lemillermicrosoft enabled auto-merge (squash) June 12, 2023 20:04
@github-actions github-actions bot removed the python Pull requests for the Python Semantic Kernel label Jun 12, 2023
@lemillermicrosoft lemillermicrosoft merged commit 3dce34c into microsoft:main Jun 13, 2023
15 checks passed
shawncal pushed a commit to shawncal/semantic-kernel that referenced this pull request Jul 6, 2023
### Description
Consistently use `useAzureOpenAI = False` in Python Notebooks. I
stumbled over this. Default was always false.
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.

None yet

3 participants