Skip to content

docs: correct Foundry model deployment example to gpt-4o-mini#38

Closed
satodayo wants to merge 1 commit into
microsoft:mainfrom
satodayo:fix/foundry-model-deployment-name-example
Closed

docs: correct Foundry model deployment example to gpt-4o-mini#38
satodayo wants to merge 1 commit into
microsoft:mainfrom
satodayo:fix/foundry-model-deployment-name-example

Conversation

@satodayo

Copy link
Copy Markdown

Summary

  • correct the Foundry model deployment example in the Episode 1 cookbook
  • use gpt-4o-mini consistently with the rest of the notebook and environment variable examples

Testing

  • not run (documentation-only change)

@farzad528

Copy link
Copy Markdown
Collaborator

Thanks @satodayo - this is a valid catch, and the inconsistency was still present (Episode 1 had gpt-4o in the env example while the rest of the notebook, the README, and the default deployment all use gpt-4o-mini).

Your branch predates the repo restructure under Foundry-IQ/, so it can no longer merge against the moved path. I have carried your exact change into #82 and credited you as a co-author. Closing this in favor of that PR - thank you for the contribution!

@farzad528 farzad528 closed this Jun 24, 2026
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