Skip to content

Fix NameError when running example code from readme - #512

Closed
ngoanpv wants to merge 1 commit into
openai:mainfrom
ngoanpv:main
Closed

ngoanpv wants to merge 1 commit into
openai:mainfrom
ngoanpv:main

Conversation

@ngoanpv

@ngoanpv ngoanpv commented Jun 30, 2023

Copy link
Copy Markdown

Replace the variable completion with chat_completion to fix the NameError.

@EliahKagan

EliahKagan commented Jul 9, 2023 •

Copy link
Copy Markdown

Issue #522 has been opened for the documentation bug that would be fixed by merging this PR or #501.

#501 was opened earlier (though this one is a bit easer to find because of its title). Probably one should be merged and the other closed as a duplicate.

I think it may make sense to also change the comment, for consistency with the preceding (non-Azure) example.

@Alisultani1

Alisultani1 commented Jul 9, 2023 via email

Copy link
Copy Markdown

@logankilpatrick

Copy link
Copy Markdown
Contributor

This is now resolved!

safa0 pushed a commit to safa0/openai-agents-python that referenced this pull request Apr 27, 2025
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.

4 participants