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

fix: fix langchain error handling to avoid crash on openai>1.0.0 #2307

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

lhrotk
Copy link
Contributor

@lhrotk lhrotk commented Nov 5, 2024

Related Issues/PRs

N/A

What changes are proposed in this pull request?

The code referencing openai<1.0 classes can't run when you have openai>1.0.

How is this patch tested?

Manual tested

Does this PR change any dependencies?

  • No. You can skip this section.
  • Yes. Make sure the dependencies are resolved correctly, and list changes here.

Does this PR add a new feature? If so, have you added samples on website?

  • No. You can skip this section.
  • Yes. Make sure you have added samples following below steps.

@lhrotk
Copy link
Contributor Author

lhrotk commented Nov 5, 2024

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@lhrotk
Copy link
Contributor Author

lhrotk commented Nov 5, 2024

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@lhrotk lhrotk changed the title WIP: make langchain error handling support openai>1.0 feat: make langchain error handling support openai>1.0 Nov 5, 2024
@lhrotk lhrotk changed the title feat: make langchain error handling support openai>1.0 fix: make langchain error handling support openai>1.0 Nov 5, 2024
@lhrotk lhrotk changed the title fix: make langchain error handling support openai>1.0 fix: fix langchain error handling to avoid crash on openai>1.0.0 Nov 5, 2024
@lhrotk
Copy link
Contributor Author

lhrotk commented Nov 6, 2024

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

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