Skip to content

Fix model name in the notebook#71

Merged
pamelafox merged 1 commit intomainfrom
fixmodelname
Oct 14, 2025
Merged

Fix model name in the notebook#71
pamelafox merged 1 commit intomainfrom
fixmodelname

Conversation

@pamelafox
Copy link
Contributor

Purpose

Fix model_name variable. The other changes are just different output from LLM, no functional changes.

Does this introduce a breaking change?

[ ] Yes
[X] No

Pull Request Type

What kind of change does this Pull Request introduce?

[X] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

@pamelafox pamelafox requested a review from Copilot October 14, 2025 19:41
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes a model name variable bug in the chat vision notebook. Previously, the code was incorrectly using os.environ["OPENAI_MODEL"] instead of the model_name variable that was properly defined earlier in the notebook.

  • Fixed variable reference from environment variable to proper model_name variable
  • Updated LLM output examples throughout notebooks (non-functional changes)
  • Updated Python version metadata in notebook

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
notebooks/chat_vision.ipynb Fixed model variable reference and updated LLM output examples
notebooks/chat_pdf_images.ipynb Updated LLM output examples and package installation logs

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@pamelafox pamelafox merged commit 51e63b2 into main Oct 14, 2025
6 checks passed
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