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

Switch from conda to uv #157

Merged
merged 28 commits into from
Jan 21, 2025
Merged

Switch from conda to uv #157

merged 28 commits into from
Jan 21, 2025

Conversation

jpt-sn
Copy link
Collaborator

@jpt-sn jpt-sn commented Dec 31, 2024

uv manages our dev and 2 optional dependencies.

Simply run:

  • uv sync --all-extras (create a .venv and install all requirements from pyproject.toml)
  • uv run -m ... (like python -m ...)

This will make easy to update dependencies:

  • uv pip list --outdated (visualize outdated dependencies)
  • uv sync --upgrade (upgrade all dependencies)
  • uv sync --all-extras --upgrade-package accelerate==1.2.0 (update a specific library to a specific version)
  • uv tree

The pyproject.toml was updated using these commands (and then manually relaxing versioning):

  • uv add -r requirements.txt
  • uv add -r requirements.dev.txt --dev
  • uv add -r requirements.converters.txt --optional converters
  • uv add -r requirements.finetune.txt --optional finetune

TODO:

  • update makefile
  • update ci
  • update cd: uv build
  • Update README

@jpt-sn jpt-sn requested a review from ollmer December 31, 2024 22:47
pyproject.toml Show resolved Hide resolved
@jpt-sn jpt-sn marked this pull request as ready for review January 20, 2025 21:05
@jpt-sn jpt-sn requested a review from ollmer January 20, 2025 21:05
README.md Outdated Show resolved Hide resolved
pyproject.toml Show resolved Hide resolved
tapeagents/llms.py Outdated Show resolved Hide resolved
Copy link
Collaborator

@ollmer ollmer left a comment

Choose a reason for hiding this comment

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

Let's update the base image for CI actions and we're good to go.

Copy link
Collaborator

@ollmer ollmer left a comment

Choose a reason for hiding this comment

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

LGTM, great work!

@jpt-sn jpt-sn merged commit 4dcf20e into main Jan 21, 2025
5 checks passed
@jpt-sn jpt-sn deleted the uv branch January 21, 2025 21:13
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