Support `>1.0` for OpenAI integration/package

strickvl commented 10 months ago

Open Source Contributors Welcomed!

Please comment below if you would like to work on this issue!

Contact Details [Optional]

support@zenml.io

What happened?

The current openai integration in ZenML is pinned to versions below 1.0.0. However, OpenAI has recently released a significant update (version 1.0.0 and above), which includes major changes. To stay up-to-date and take advantage of the new features and improvements, ZenML needs to support the latest OpenAI versions. This update might necessitate a breaking change in ZenML, shifting to support only versions >1.0.0 of the OpenAI integration.

Task Description

Update the openai integration within ZenML to support versions >=1.0.0. This task involves:

Unpinning the current version restriction in src/zenml/integrations/openai/init.py.
Ensuring compatibility with the new OpenAI version 1.0.0 and above.
Making necessary adjustments or rewrites in the ZenML codebase to accommodate any breaking changes introduced in the new OpenAI version.
Updating documentation and examples to reflect the new integration version support.
Considering making this update a breaking change for ZenML, where only OpenAI versions >1.0.0 will be supported going forward.

Expected Outcome

Successful integration of OpenAI version 1.0.0 and above in ZenML, with all tests passing and documentation updated to guide users on the new integration version.

Additional context

This update is crucial to keep ZenML integrations current and to provide users with the latest features and improvements from OpenAI.

Code of Conduct

[ ] I agree to follow this project's Code of Conduct

Senthi1Kumar commented 9 months ago

Hi @strickvl, should I work on this issue

strickvl commented 9 months ago

@Senthi1Kumar if you'd like to! Please make sure to follow our contributors guide, esp by branching off and making your PR to the develop branch. Let us know if you have further questions or if you'd like to discuss your approach etc.

remiBoudreau commented 8 months ago

Having ridiculous trouble getting this to work with all the dependencies resolved... when i run

zenml integration install -y -i feast

or

pip install docs/.requirements-docs-frozen.txt

using the Docker container for zenml-dev.Dockerfile, the dependencies become impossible to resolve. If I change the versions for packages with conflicting dependencies, it creates a chain of more packages with conflicting dependencies...

strickvl commented 8 months ago

You can safely ignore feast and the requirements in pip install docs/.requirements-docs-frozen.txt.

On Fri, 9 Feb 2024 at 23:35, Jean-Michel Remi Boudreau < @.***> wrote:

Having ridiculous trouble getting this to work with all the dependencies resolved... when i run

zenml integration install -y -i feast

or

pip install docs/.requirements-docs-frozen.txt

using the Docker container for zenml-dev.Dockerfile the dependencies become impossible to resolve

— Reply to this email directly, view it on GitHub https://github.com/zenml-io/zenml/issues/2210#issuecomment-1936693185, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAZRNJX72LJQDWX2PYPUAFDYS2QDJAVCNFSM6AAAAABBLHJXLCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMZWGY4TGMJYGU . You are receiving this because you were mentioned.Message ID: @.***>

strickvl commented 8 months ago

Actually, now that I took a closer look, this issue/task is blocked until we upgrade our Pydantic dependency to >2.0.0, since OpenAI pin this to v2.4.2. We're starting to work on this internally, but I'd hold off on this for the moment as it's currently impossible.

remiBoudreau commented 8 months ago

Okay no worries. Thanks for giving me the heads up :)

zenml-io / zenml