Skip to content

Fix broken openai import in Azure_OpenAI_Graders notebook - #291

Merged
Hanchi Wang (luigiw) merged 2 commits into
mainfrom
copilot/fix-outdated-dependency
Mar 17, 2026
Merged

Hanchi Wang (luigiw) merged 2 commits into
mainfrom
copilot/fix-outdated-dependency

Conversation

Copilot AI commented Mar 2, 2026 •

Copy link
Copy Markdown
Contributor

openai v2.x moved EvalStringCheckGrader out of openai.types into openai.types.graders, renaming it to StringCheckGrader, causing a ModuleNotFoundError on notebook startup.

Changes

  • Import: openai.types.eval_string_check_grader.EvalStringCheckGrader → openai.types.graders.string_check_grader.StringCheckGrader
  • Usage: Renamed all references from EvalStringCheckGrader to StringCheckGrader
# Before
from openai.types.eval_string_check_grader import EvalStringCheckGrader
oai_string_check_grader = EvalStringCheckGrader(...)

# After
from openai.types.graders.string_check_grader import StringCheckGrader
oai_string_check_grader = StringCheckGrader(...)

Constructor signature is unchanged.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: w-javed <7674577+w-javed@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix outdated dependency for EvalStringCheckGrader Fix broken openai import in Azure_OpenAI_Graders notebook Mar 2, 2026
@w-javed
Waqas Javed (w-javed) marked this pull request as ready for review March 16, 2026 19:42
@w-javed
Waqas Javed (w-javed) requested a review from a team as a code owner March 16, 2026 19:42
Copilot AI had a problem deploying to external-contribution March 17, 2026 06:49 Failure

This branch had an error being deployed

1 failed deployment
external-contribution — ebfe5b10 Deployed Mar 17, 2026 by Copilot via run-samples #314
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.

3 participants