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

[ci] Revert UV pip compile for LLM requirements #51118

Merged
merged 1 commit into from
Mar 6, 2025
Merged

Conversation

khluu
Copy link
Contributor

@khluu khluu commented Mar 6, 2025

UV doesn't seem to carry over the right environment markers in requirements_compiled.txt. We can temporarily revert this until we find a fix for the issue.

@khluu khluu requested review from kouroshHakha and aslonnie March 6, 2025 07:12
@aslonnie aslonnie added the go add ONLY when ready to merge, run all tests label Mar 6, 2025
@aslonnie aslonnie merged commit cad5fe7 into master Mar 6, 2025
5 checks passed
@aslonnie aslonnie deleted the khluu/revertuvllm branch March 6, 2025 17:57
elimelt pushed a commit to elimelt/ray that referenced this pull request Mar 9, 2025
UV doesn't seem to carry over the right environment markers in
`requirements_compiled.txt`. We can temporarily revert this until we
find a fix for the issue.

Signed-off-by: Kevin H. Luu <[email protected]>
park12sj pushed a commit to park12sj/ray that referenced this pull request Mar 18, 2025
UV doesn't seem to carry over the right environment markers in
`requirements_compiled.txt`. We can temporarily revert this until we
find a fix for the issue.

Signed-off-by: Kevin H. Luu <[email protected]>
jaychia pushed a commit to jaychia/ray that referenced this pull request Mar 19, 2025
UV doesn't seem to carry over the right environment markers in
`requirements_compiled.txt`. We can temporarily revert this until we
find a fix for the issue.

Signed-off-by: Kevin H. Luu <[email protected]>
Signed-off-by: Jay Chia <[email protected]>
jaychia pushed a commit to jaychia/ray that referenced this pull request Mar 19, 2025
UV doesn't seem to carry over the right environment markers in
`requirements_compiled.txt`. We can temporarily revert this until we
find a fix for the issue.

Signed-off-by: Kevin H. Luu <[email protected]>
Signed-off-by: Jay Chia <[email protected]>
Drice1999 pushed a commit to Drice1999/ray that referenced this pull request Mar 23, 2025
UV doesn't seem to carry over the right environment markers in
`requirements_compiled.txt`. We can temporarily revert this until we
find a fix for the issue.

Signed-off-by: Kevin H. Luu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
go add ONLY when ready to merge, run all tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants