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

Update usage of OpenAI client #166

Merged
merged 1 commit into from
Apr 28, 2024
Merged

Conversation

AbhinavMir
Copy link
Contributor

@AbhinavMir AbhinavMir commented Apr 27, 2024

📥 Pull Request

📘 Description
The Jupyter notebook example currently uses an outdated OpenAI Python library example.

🎯 Goal
Update the usage to reference OpenAI's client correctly.

@AbhinavMir AbhinavMir marked this pull request as ready for review April 28, 2024 02:04
@areibman
Copy link
Contributor

Technically the same but I think adding the api key in the line adds clarity. So LGTM

@AbhinavMir
Copy link
Contributor Author

AbhinavMir commented Apr 28, 2024

@areibman Hm, weird. Current usage was erroring out with OpenAIError: The api_key client option must be set either by passing api_key to the client or by setting the OPENAI_API_KEY environment variable for me. I was running off the same example mentioned in the docs. Thanks nonetheless! Thought I'd send out what worked for me!

@areibman areibman merged commit 712bb68 into AgentOps-AI:main Apr 28, 2024
2 checks passed
@areibman
Copy link
Contributor

Thank you @AbhinavMir!!

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