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

Fixed tpyos, added some instructions, and fixed an AI example. #3342

Merged
merged 1 commit into from
Nov 27, 2023

Conversation

IntegerMan
Copy link
Contributor

I went over the AI samples yesterday and noticed a few tpyos. This commit addresses those.

Additionally, I got tripped up in a few places because I didn't customize the deployment name. Even though the getting started notebook talks about this, I thought it'd be better to add a brief reminder on this where relevant.

Finally, I noticed an error with one of the notebooks where the prompts.Last would result in a sequence contains no elements LINQ error with the sequence of cells as currently written. I pulled the cell to track prompts up higher in the notebook so the prompts in that section get added to the list before the .Last call is made.

Also, I love these examples and what's possible with Polyglot and AI; this makes it easy to demonstrate / teach AI concepts, semantic kernel, RAG, and general OpenAI integration. Beyond that, though, I think this is now my preferred way of working with DALL-E. Also: kudos to you on the Hello DALL-E reference.

@IntegerMan IntegerMan changed the title Fixed tpyos and added some instructions and fixed an AI example. Fixed tpyos, added some instructions, and fixed an AI example. Nov 25, 2023
@colombod colombod added the Area-Documentation Improvements or additions to documentation label Nov 27, 2023
@colombod colombod merged commit 415934a into dotnet:main Nov 27, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants