-
Notifications
You must be signed in to change notification settings - Fork 7.5k
[ci][deps][1/3] PY313 DEP UNIFICATION: compiling requirements_compiled_py3.13.txt and depsets #62864
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][deps][1/3] PY313 DEP UNIFICATION: compiling requirements_compiled_py3.13.txt and depsets #62864
Changes from all commits
7fb856f
476497e
50aff7a
4ceb466
17dfa13
1f72736
7dd2cff
2c26cdb
f209a0e
93802b9
298ee7b
371d0e1
e195c89
943f417
c7976d1
fc82923
61b7103
087c9c9
da050d6
0584e88
5dec002
47936c3
06ba985
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -79,6 +79,76 @@ compile_pip_dependencies() { | |
| ) | ||
| } | ||
|
|
||
| compile_313_pip_dependencies() { | ||
| # Compile boundaries for the py3.13 dependency set. Uses the py313 override | ||
| # directories (python/requirements/py313/ and python/requirements/ml/py313/) | ||
| # where available, and falls back to the shared requirement files otherwise. | ||
| TARGET="${1-requirements_compiled_py3.13.txt}" | ||
|
|
||
| if [[ "${HOSTTYPE}" == "aarch64" || "${HOSTTYPE}" = "arm64" ]]; then | ||
| # Resolution currently does not work on aarch64 as some pinned packages | ||
| # are not available. Once they are reasonably upgraded we should be able | ||
| # to enable this here. | ||
| echo "Skipping for aarch64" | ||
| return 0 | ||
| fi | ||
|
|
||
| ( | ||
| # shellcheck disable=SC2262 | ||
| alias pip="python -m pip" | ||
|
|
||
| cd "${WORKSPACE_DIR}" | ||
|
|
||
| echo "Target file: $TARGET" | ||
| pip install "pip-tools==7.4.1" "wheel==0.45.1" | ||
|
|
||
| # Required packages to lookup e.g. dragonfly-opt | ||
| HAS_TORCH=0 | ||
| python -c "import torch" 2>/dev/null && HAS_TORCH=1 | ||
| pip install --no-cache-dir numpy torch | ||
|
|
||
| pip-compile --verbose --resolver=backtracking \ | ||
| --pip-args --no-deps --strip-extras --no-header \ | ||
| --unsafe-package ray \ | ||
| --unsafe-package pip \ | ||
| --unsafe-package setuptools \ | ||
| -o "python/$TARGET" \ | ||
| python/requirements.txt \ | ||
| python/requirements/lint-requirements.txt \ | ||
| python/requirements/py313/test-requirements.txt \ | ||
| python/requirements/cloud-requirements.txt \ | ||
| python/requirements/docker/ray-docker-requirements.txt \ | ||
| python/requirements/ml/py313/core-requirements.txt \ | ||
| python/requirements/ml/py313/data-requirements.txt \ | ||
| python/requirements/ml/py313/data-test-requirements.txt \ | ||
| python/requirements/ml/py313/dl-cpu-requirements.txt \ | ||
| python/requirements/ml/py313/ml-requirements.txt \ | ||
| python/requirements/ml/py313/third_party.txt \ | ||
| python/requirements/ml/py313/rllib-requirements.txt \ | ||
| python/requirements/ml/py313/rllib-test-requirements.txt \ | ||
| python/requirements/ml/py313/train-requirements.txt \ | ||
| python/requirements/ml/py313/train-test-requirements.txt \ | ||
| python/requirements/ml/py313/tune-requirements.txt \ | ||
| python/requirements/ml/py313/tune-test-requirements.txt \ | ||
| python/requirements/security-requirements.txt | ||
|
elliot-barn marked this conversation as resolved.
|
||
|
|
||
| # Delete local installation | ||
| sed -i "/@ file/d" "python/$TARGET" | ||
|
|
||
| # Remove +cpu and +pt20cpu suffixes e.g. for torch dependencies | ||
| # This is needed because we specify the requirements as torch==version, but | ||
| # the resolver adds the device-specific version tag. If this is not removed, | ||
| # pip install will complain about irresolvable constraints. | ||
| sed -i -E 's/==([\.0-9]+)\+[^\b]*cpu/==\1/g' "python/$TARGET" | ||
|
elliot-barn marked this conversation as resolved.
|
||
|
|
||
| cat "python/$TARGET" | ||
|
|
||
| if [[ "$HAS_TORCH" == "0" ]]; then | ||
| pip uninstall -y torch | ||
| fi | ||
| ) | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Duplicated compile function risks future divergenceLow Severity
Reviewed by Cursor Bugbot for commit fc82923. Configure here. |
||
| } | ||
|
|
||
| test_cpp() { | ||
| if [[ "${OSTYPE}" == darwin* ]]; then | ||
| echo "use macos_ci.sh to run cpp tests" | ||
|
|
||


Uh oh!
There was an error while loading. Please reload this page.