Skip to content

Utilize uv for dependencies, and re-use containers where possible#160

Open
M1kep wants to merge 2 commits into
mc-bench:mainfrom
M1kep:m1kep/deps/use-uv
Open

Utilize uv for dependencies, and re-use containers where possible#160
M1kep wants to merge 2 commits into
mc-bench:mainfrom
M1kep:m1kep/deps/use-uv

Conversation

@M1kep

@M1kep M1kep commented Mar 17, 2025

Copy link
Copy Markdown
Contributor

No description provided.

Comment on lines -34 to +30
celery[redis]==5.4.0
celery==5.4.0

@M1kep M1kep Mar 17, 2025

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

See astral-sh/uv#1595 for rational and some background. TLDR: It shouldn't make a difference.

There's some more chatter here too(Is also linked in the uv issue above) - jazzband/pip-tools#1613

@huntcsg

huntcsg commented Mar 17, 2025

Copy link
Copy Markdown
Contributor

Is this just a performance improvement? UV has better dependency resolution?
This change seems fine. I just want to understand the benefit better.

@M1kep

M1kep commented Mar 17, 2025

Copy link
Copy Markdown
Contributor Author

Yeah, mostly performance. I think their dependency resolution has some more features, but I don't think they'll be of use in this project.
10x-100x according to the uv team - https://github.com/astral-sh/uv/blob/main/BENCHMARKS.md

uv performance improvements are more obvious in installation flows I believe, so would be curious to see the image build speedup by switching as well, but will keep that for later

@IsaacGemal

Copy link
Copy Markdown
Contributor

I like UV a lot

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