Skip to content

Add openai/datalib/__init__.py #404

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

Merged
merged 1 commit into from
Apr 17, 2023
Merged

Add openai/datalib/__init__.py #404

merged 1 commit into from
Apr 17, 2023

Conversation

harupy
Copy link
Contributor

@harupy harupy commented Apr 17, 2023

Fixes #403

Test

FROM python:3.8

RUN pip install git+https://github.com/harupy/[email protected]
RUN python -c 'from openai.api_resources.embedding import Embedding'

works fine now

Copy link
Collaborator

@hallacy hallacy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@hallacy hallacy merged commit c5f0cc4 into openai:main Apr 17, 2023
davedittrich pushed a commit to davedittrich/openai-python that referenced this pull request Nov 14, 2023
cgayapr pushed a commit to cgayapr/openai-python that referenced this pull request Dec 14, 2024
safa0 pushed a commit to safa0/openai-agents-python that referenced this pull request Apr 27, 2025
## Summary:
Towards openai#404. I made this configurable because it's not clear this is
always a good thing to do. I also made it default to False because I'm
not sure if this could cause errors.

If it works out well, we can switch the default in the future as a small
breaking changes

## Test Plan:
Unit tests
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.

ModuleNotFoundError: No module named 'openai.datalib' on main branch
2 participants