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

Migration from Poetry to uv #218

Merged
merged 16 commits into from
Oct 8, 2024
Merged

Migration from Poetry to uv #218

merged 16 commits into from
Oct 8, 2024

Conversation

ProKil
Copy link
Member

@ProKil ProKil commented Oct 7, 2024

uv is a new python dependency manager which is much faster than poetry, and supports the standard Python pyproject file. This becomes the choices of several new Python packages and many existing ones are also migrating.

The changes to the sotopia API and dependency is minimal. The only change is that install locally from source no long requires poetry. One can use pip install -e .[<extras>] instead of poetry install. One can also opt in uv for faster installation through uv run python ..., or uv sync; source .venv/bin/activate (not recommended)

📑 Description

  1. Update pyproject.toml
  2. Change CI
  3. Change docs

✅ Checks

  • My pull request adheres to the code style of this project
  • My code requires changes to the documentation
  • I have updated the documentation as required
  • All the tests have passed
  • Branch name follows type/descript (e.g. feature/add-llm-agents)
  • Ready for code review

ℹ Additional Information

Copy link

codecov bot commented Oct 7, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 71.37%. Comparing base (71d6199) to head (66ba0ae).
Report is 1 commits behind head on main.

@@            Coverage Diff             @@
##             main     #218      +/-   ##
==========================================
- Coverage   71.40%   71.37%   -0.04%     
==========================================
  Files          55       55              
  Lines        2868     2868              
==========================================
- Hits         2048     2047       -1     
- Misses        820      821       +1     

see 1 file with indirect coverage changes

@ProKil ProKil requested a review from XuhuiZhou October 7, 2024 05:54
Copy link
Member

@XuhuiZhou XuhuiZhou left a comment

Choose a reason for hiding this comment

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

any chance you can sync the documentation webpage?
As well as telling people which command to run to install the extra packages?

@ProKil
Copy link
Member Author

ProKil commented Oct 8, 2024

@XuhuiZhou I have updated the docs and provided the guide for using uv.

@ProKil ProKil merged commit ed626ec into main Oct 8, 2024
8 checks passed
@ProKil ProKil deleted the feature/migrate-to-uv branch October 9, 2024 22:41
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.

2 participants