Conversation
|
note: no need to change the tutorial notebooks code, since notebooks do not use uv. the notebooks will continue using pip install as currently |
| ] | ||
|
|
||
| [tool.uv] | ||
| conflicts = [ |
There was a problem hiding this comment.
Is there a better way to handle the conflicts? see my question in the main discussion regarding that
|
One unique aspect of granite-switch is the fact that it's an infra library and should work with different environments.
|
just tested the uv branch on CCC and it worked out of the box. as a matter of fact the non uv branch did not work for me out of the box on CCC. But i only tried the HF dependency, not vllm. |
My suggestion is this:
So [tool.uv] goes from 6 conflict pairs down to 1: [dependency-groups] [tool.uv] |
|
@aviv1ron1 Thanks for looking into it. Couple of thoughts:
|
|
@antonpibm I am addressing all you comments: What this branch changesMigrates dependency management from pip extras to uv dependency groups in Before: pip install -e ".[hf,vllm,compose]"After: uv sync --group dev # vllm 0.19.x
uv sync --group dev-vllm20 # vllm 0.20.xWhat was testedThe full test suite was run on a GPU cluster (4× GPU, Python 3.12) against both supported vllm versions, using
Known failure
The failure is not related to the ConclusionThe The |
This PR switches the project from pip/venv to uv for dependency management.
What changed:
CUDA note: the vllm/vllm20 extra split already handles the CUDA 12 vs 13 version requirement — the lockfile is universal and does not tie to a specific CUDA environment.