Skip to content

Commit ee7ab31

Browse files
authored
chore: migrate openai dependency from 1.x to 2.x (#1849)
- Relax openai version constraint from ~=1.106 (>=1.106, <2.0) to ~=2.0 (>=2.0, <3.0) across all packages that directly pin it. - Remove explicit openai dep in favor of transitive resolution for agno and related packages - Bump openpipe-art from ==0.5.4 to ~=0.5.17 for openai 2.x compatibility, and remove the torchtune_args field which was dropped in openpipe-art 0.5.9. - Regenerate affected uv.lock files (root, agno, openpipe-art, agno_personal_finance, multi_frameworks, rl_with_openpipe_art, agents). Output of `./ci/scripts/license_diff.py` ``` Added packages: - polars 1.39.3 — MIT - polars-runtime-32 1.39.3 — MIT - setproctitle 1.3.7 — BSD-3-Clause Removed packages: - litellm 1.74.1 (duplicate version removed, 1.74.9 remains) - mem0ai 0.1.118 (duplicate version removed, 0.1.115 remains) Changed packages: - instructor 1.12.0 → 1.15.1 - openai 1.109.1 → 2.30.0 - openpipe-art 0.5.4 → 0.5.17 - strands-agents 1.27.0 → 1.34.1 ``` Closes ## By Submitting this PR I confirm: - I am familiar with the [Contributing Guidelines](https://github.com/NVIDIA/NeMo-Agent-Toolkit/blob/develop/docs/source/resources/contributing/index.md). - We require that all contributors "sign-off" on their commits. This certifies that the contribution is your original work, or you have rights to submit it under the same license, or a compatible license. - Any contribution which contains commits that are not Signed-Off will not be accepted. - When the PR is ready for review, new or existing tests cover these changes. - When the PR is ready for review, the documentation is up to date with these changes. ## Summary by CodeRabbit * **Chores** * Simplified dependency management by using package extras instead of direct pins and removed redundant explicit pins. * Updated openpipe-art to a more flexible compatible release range. * **Refactor** * Removed an unused configuration field from the model training configuration. * **Tests** * Adjusted tests to reflect the removed configuration field. Authors: - Will Killian (https://github.com/willkill07) Approvers: - Anuradha Karuppiah (https://github.com/AnuradhaKaruppiah) - https://github.com/Salonijain27 URL: #1849
1 parent 9672dd1 commit ee7ab31

14 files changed

Lines changed: 549 additions & 439 deletions

File tree

examples/agents/uv.lock

Lines changed: 6 additions & 90 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)