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

Refactor OpenAIEngine to use the openai-python package #31

Closed
zhudotexe opened this issue Feb 23, 2024 · 0 comments · Fixed by #38
Closed

Refactor OpenAIEngine to use the openai-python package #31

zhudotexe opened this issue Feb 23, 2024 · 0 comments · Fixed by #38
Assignees
Labels
enhancement New feature or request

Comments

@zhudotexe
Copy link
Owner

In the current version of Kani we don't use the official OpenAI package, instead calling the API ourselves with an aiohttp client. I chose to do this because in the earlier 0.x versions, the OpenAI package did not yet support async HTTP requests and had a weird module-level structure. Since the release of their 1.x version, these concerns are no longer valid, and we should move to using the official package to decrease code fragility.

@zhudotexe zhudotexe added the enhancement New feature or request label Feb 23, 2024
@zhudotexe zhudotexe self-assigned this Feb 23, 2024
@zhudotexe zhudotexe mentioned this issue Apr 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant