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

inference job failed #424 #445

Closed
wants to merge 1 commit into from

Conversation

kamtschatka
Copy link
Collaborator

added handling for the case where the chatCompletion is returned as a string

NOTE:

  • I can not reproduce this locally, but I worked with @hujh818 over discord who helped me figure out what is going on
  • I have absolutely no idea, why that is a string, but the logs he added, clearly showed that it is a string and after adding a JSON.parse, he confirmed that it is working now...

added handling for the case where the chatCompletion is returned as a string
@MohamedBassem
Copy link
Collaborator

Not a big fan of this at all to be honest. The best we can do is update the openai package and hope it resolves the issue. If it doesn't, then they should take it with the API provider. Upgraded openai here b56655c

@kamtschatka kamtschatka deleted the inference-issues branch October 5, 2024 13:40
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