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

Fix autogen versions in pyproject.toml file for studio #5023

Closed
wants to merge 3 commits into from

Conversation

ekzhu
Copy link
Collaborator

@ekzhu ekzhu commented Jan 13, 2025

Bug. Try the following:

uv venv --python=3.11
source .venv/activate/bin
uv pip install -U autogenstudio
autogenstudio ui

This will take you to the older version with green UI.

If you do the following:

uv pip install -U autogenstudio==0.4.0.1

You get the following error:

(temp) (base) ekzhu@xxx temp % uv pip install autogenstudio==0.4.0.1
  × No solution found when resolving dependencies:
  ╰─▶ Because there is no version of autogen-core==0.4.0.dev13 and autogenstudio==0.4.0.1 depends on autogen-core==0.4.0.dev13,
      we can conclude that autogenstudio==0.4.0.1 cannot be used.
      And because you require autogenstudio==0.4.0.1, we can conclude that your requirements are unsatisfiable.

      hint: `autogen-core` was requested with a pre-release marker (e.g., autogen-core==0.4.0.dev13), but pre-releases weren't
      enabled (try: `--prerelease=allow`)

Seems the deployed version still depends on dev13? This PR to make the version range rather pining on a specific version, to avoid future issues.

@ekzhu ekzhu requested a review from victordibia January 13, 2025 07:38
Copy link

codecov bot commented Jan 13, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 68.53%. Comparing base (70f7e99) to head (e1e8592).
Report is 4 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5023      +/-   ##
==========================================
+ Coverage   68.50%   68.53%   +0.02%     
==========================================
  Files         156      156              
  Lines       10129    10140      +11     
==========================================
+ Hits         6939     6949      +10     
- Misses       3190     3191       +1     
Flag Coverage Δ
unittests 68.53% <ø> (+0.02%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jackgerrits jackgerrits added this to the 0.4.1 milestone Jan 13, 2025
@victordibia
Copy link
Collaborator

Taking a look.

@victordibia
Copy link
Collaborator

Thanks,
I think the issue was just that the last release did not use the right deps (dev0.13. ,...), meaning pyproject.toml in its current form is not the issue.

Can you confirm you still get errors with the below.

uv venv --python=3.11
source .venv/activate/bin
uv pip install -U autogenstudio
autogenstudio ui
uv pip install -U autogenstudio==0.4.0.1

@ekzhu
Copy link
Collaborator Author

ekzhu commented Jan 13, 2025

Still getting error on this:

uv pip install -U autogenstudio==0.4.0.1

But this is because the version?

uv pip install -U autogenstudio==0.4.0.2

Works without error.

@victordibia
Copy link
Collaborator

victordibia commented Jan 13, 2025

Yes, the 0.4.0.1 has a bug in pyproject.toml.
We probably can close this issue.

@ekzhu ekzhu closed this Jan 14, 2025
@ekzhu ekzhu deleted the fix-studio-pyproject-toml branch January 14, 2025 01:47
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.

3 participants