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

Mismatch between old OpenAI API in bertopic/backend/_openai and current OpenAI (v1.33.0) #2044

Open
abeusher opened this issue Jun 9, 2024 · 1 comment

Comments

@abeusher
Copy link

abeusher commented Jun 9, 2024

Trying a simple import (from bertopic import BERTopic) breaks due to a difference between the old openai API (when the bertopic/backend/_openai was originally written) and the current openai API (version 1.33.0).

I implemented a fix in my own branch of the code here:

https://github.com/abeusher/BERTopic/blob/bugfix/new_openai_api/bertopic/backend/_openai.py

I am happy to provide a pull request when you are ready.

@MaartenGr
Copy link
Owner

Ah yes, it seems that the backend is not yet updated whereas the representation model is.

A PR would be appreciated! Do note though that it needs to take the same structure as the representation model, so passing the entire client and not the API key.

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

No branches or pull requests

2 participants